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

Report #826

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Report #826

wants to merge 60 commits into from

Conversation

HusseinSarrar05
Copy link

No description provided.

HusseinSarrar05 and others added 30 commits June 10, 2024 13:40
- Implemented django mock tests for assumed branch cases on tests.py in web subdirectory of mygpo.
- Need to clarify, and add better manual branch instrumentation, for proper testing and coverage enhancment
- Completed manual instrumentation over branches, and enhanced testing in tests.py for coverage enhancement
spower77 and others added 30 commits June 24, 2024 22:23
Adjusted format for test implementation, instrumentation, and coverage to match team formatting, and specific output to a text file.
Final major adjustment to episode_status_icon testing. Changed coverage.txt file location to be a universal project root, rather than a local directory.

Full branch coverage achieved
Adjusted testing and instrumentation with additional helper methods to report coverage information to root
…ctions episode_status_icon and mark_outdated in class EpisodeUpdater

Completed instrumentation, testing and coverage, with coverage reporting functionality.

I have added additional instrumentation for hidden branches, and implicit branches.

Branch instrumentation for EpisodeUpdater was only applied to mark_outdated as they had the only functional instrumentable branches in the class, with tests checking functionality of Updater, and also mark_outdated for proper functionality in Django.

Please note, coverage.txt will be generated in the same directory of the test directory, regardless of local system. Please adjust as necessary to conform to team standards.
Had to change exporting behaviour for coverage reporting in accordance to team standards
…odeUpdater)

- Modified branch naming conventions
- Re-analysed and found instrumentatino and testing errors with EpisodeUpdater testing
[SAMUEL] 100% coverage on both function's tests
[ANDREAS] 100% coverage on both function's tests
[MOHAMED] 100% coverage on both function's tests
[HUSSEIN] 100% coverage on both function's tests
- Added member contributions to coverage and testing sections.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants