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

Finish all open issues and hand over work before hiatus #459

Closed
SebastianZimmeck opened this issue Apr 16, 2024 · 5 comments
Closed

Finish all open issues and hand over work before hiatus #459

SebastianZimmeck opened this issue Apr 16, 2024 · 5 comments
Assignees
Labels
organization An issue about organizing our work

Comments

@SebastianZimmeck
Copy link
Member

@sophieeng, as discussed, since you will be on hiatus after this semester, it would be good if you can check if there is anything open on your end, and hand over any open work. Especially, it would be good if you can explain to @Mattm27 and @franciscawijaya, the unit testing and continuous integration.

I am opening this issue here, but it also relates to the crawler or any other repo you have worked on.

@SebastianZimmeck SebastianZimmeck added the organization An issue about organizing our work label Apr 16, 2024
@SebastianZimmeck
Copy link
Member Author

Not sure, for example, if the mv3 migration has any impact on the unit test suite or continuous integration. If so, we should address that before you leave (and if so, open a new issue for that).

@franciscawijaya
Copy link
Member

Reread the repo (eg. unit tests) and meet with @sophieeng

@sophieeng
Copy link
Member

The test suite seems to be running as expected in the CI (GitHub actions) and locally. The READMe explanations also seem clear to me.

I finished adding explanations to the gpc.test.js test file. I believe the other tests in the other files are straightforward to understand. The gpc.test.js file was more complicated but should be accessible now.

When I meet with Francisca and Matt later on this week, I'll walk them through all the test files in the test suite, as well as the manual UI tests. I'll add more comments to the test files and READMe as needed from whatever questions they have.

sophieeng added a commit that referenced this issue Apr 29, 2024
@SebastianZimmeck
Copy link
Member Author

Sounds good, @sophieeng!

When I meet with Francisca and Matt later on this week,

@Mattm27 and @franciscawijaya, before you meet with @sophieeng, can you run the tests on your end and make sure you know how to do the UI tests?

@sophieeng
Copy link
Member

I've met with Francisca and Matt to explain the repo + test suite. All the tests are working as expected. Don't hesitate to reach out if you have any questions about them in the future! This issue is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
organization An issue about organizing our work
Projects
None yet
Development

No branches or pull requests

3 participants