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.

Sign up for GitHub

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

Jump to bottom

DRAFT: ADD: All of testing, very large PR. #44

Closed
wants to merge 30 commits into from
Closed

DRAFT: ADD: All of testing, very large PR. #44

wants to merge 30 commits into from

Conversation

Ashassins
Copy link
Collaborator

No description provided.

emilejbm and others added 23 commits February 11, 2023 17:50
@emilejbm
ini push for visibility
e1b5ef4
@emilejbm
rebase with man
d4be788
@Ashassins
MOVE testfiles to the right folders
7944673
@Ashassins
FIX: matching package names for test files, tests now run but errors
0bdf2cd
@Ashassins
FIX: matching package names for test files, tests now run but errors,…
602515e 
… but for worker
@Ashassins
FIX: api tests now run
855db8c
@Ashassins
ADD: working restapi input tests
86aa194
@Ashassins
UPDATE: make all functions exported in api.go
4910f85
@Ashassins
ADD: TestDecodeResponse
5a7c7e2
@emilejbm
9 test cases corrected. cli test command is functional but printing o…
59429a8 
…f coverage is incorrect
@emilejbm
addition of linting on push, test command working but incorrect line …
3ec283c 
…coverage
@emilejbm
rework of manager_test. tests need environment variable to be set
6536750
@Ashassins
EDIT: Make all api.go functions exported for testing
0d894d4
@Ashassins
ADD: test cases for api. currently at 18.9% coverage
0ee4816
@Ashassins
DELETE: unneeded draft testfiles dir
42b1973
@Ashassins
ADD: Check Repo Contributors Success. 36.5% for REST API tests
620a0d3
@Ashassins
ADD REST API test coverage 54.9%: Add RepoIssueAvgLifespan
8766231
@emilejbm
slight change to lowercase function names now that tests are within p…
0e8520e 
…ackages
@emilejbm
syntax fix to yml file. had not been tested
e2fa924
@emilejbm
removal of bad error name. autograde yml test will fail because of en…
0b2210c 
…v variable
@Ashassins
ADD: rest api tests. @ 77%
4e484af
@Ashassins
FIX: remove incomplete func
1571e1a
@Ashassins
FIX: Comment out an incomplete test. rest api at 74.25
c6e377b
@Ashassins Ashassins changed the title ADD: All of testing, very large PR. DRAFT: ADD: All of testing, very large PR. Feb 12, 2023
@Ashassins Ashassins marked this pull request as draft February 12, 2023 19:50
@Ashassins Ashassins linked an issue Feb 12, 2023 that may be closed by this pull request
Add Testing #33
Open
@emilejbm emilejbm marked this pull request as ready for review February 12, 2023 22:47
@emilejbm emilejbm closed this Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Testing
2 participants
@Ashassins @emilejbm