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

Test db #184

Merged
merged 10 commits into from
Jun 13, 2024
Merged

Test db #184

merged 10 commits into from
Jun 13, 2024

Conversation

jak103
Copy link
Owner

@jak103 jak103 commented Jun 12, 2024

No description provided.

func TestPostGoalBadRequest(t *testing.T) {
app := apis.CreateTestApp()

req, _ := http.NewRequest(fiber.MethodPost, "/api/v1/goals", nil)
Copy link
Collaborator

@tannerwheeler tannerwheeler Jun 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jak103 This is where the failure is occuring.


app := apis.CreateTestApp()

req, _ := http.NewRequest(fiber.MethodPost, "/api/v1/goals", bodyReader)
Copy link
Collaborator

@tannerwheeler tannerwheeler Jun 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jak103 This is where the failure is occurring.

@funkybooboo
Copy link
Collaborator

The tests in github.com/jak103/powerplay/internal/server/apis/sports/stats have a failure.

In github.com/jak103/powerplay/backend/internal/utils/log I commented out the asserts that where giving a failure and added a todo to get them fixed.

@jak103 jak103 merged commit 58afd1c into main Jun 13, 2024
1 check passed
@jak103 jak103 deleted the test-db branch June 13, 2024 18:27
Skylude pushed a commit to tannerwheeler/powerplay-backend-stats that referenced this pull request Jun 23, 2024
* Commit for class

* Finished class demo

* Update .gitignore to ignore local.env

* Testing the DB

* Fix the build

* some fixes

* Fix failing test

---------

Co-authored-by: Jacob Christensen <[email protected]>
Co-authored-by: Nate Stott <[email protected]>
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