Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove hardcoded paths #65

Open
ktaekwon000 opened this issue Oct 7, 2020 · 1 comment
Open

Remove hardcoded paths #65

ktaekwon000 opened this issue Oct 7, 2020 · 1 comment
Labels
enhancement New feature or request priority.Medium Medium Priority

Comments

@ktaekwon000
Copy link
Collaborator

While implementing some new features that involve storage, we hardcoded the data directory into our application, making our application unable to be configured with a custom file path.
image
I don't think this is a requirement for the application (I didn't see any relevant constraints on the tP info page) but this goes against the design of the existing ab3 application.

@ktaekwon000 ktaekwon000 added enhancement New feature or request priority.Low Low priority labels Oct 7, 2020
@ktaekwon000
Copy link
Collaborator Author

In the process of coding my own tests, I found that the test code heavily depends on variable file paths, and it would be hard to write test code for the sections with hardcoded paths.

As testability is a grading criterion (I think?), I am increasing the priority of this issue.

@ktaekwon000 ktaekwon000 added priority.Medium Medium Priority and removed priority.Low Low priority labels Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority.Medium Medium Priority
Projects
None yet
Development

No branches or pull requests

1 participant