Refactor `TestGetTotalTribes` To Use A Real Postgres DB For The Test

[elraphty]

Context

Currently, we test the TestGetTotalTribes with a mock database.

Design

We want to refactor the tests to use a real Postgres DB spun up on docker, follow the instructions in the Unit Testing section of README.md to run spin up a database in docker, and change the test_config.go configuration.

After spinning up the Postgres DB container:

• Add this line of code on the first line and second of the test function TestGetTotalTribes to connect to the Test DB

  teardownSuite := SetupSuite(t)
  defer teardownSuite(t)

  The test will throw a dereference error if not connected to a DB.

• Remove all mock database functions in the TestGetTotalTribes test cases e.g. mockDb.On("GetTribesTotal", mock.Anything).Return(expectedTribesCount).
• Change the database parameter in tHandler := NewTribeHandler(mockDb) to tHandler := NewTribeHandler(db.TestDb)
• Create a tribe using the db.TestDb.CreateOrEditTribe() function e.g

  tribe := db.Tribe{
  UUID: ${tribe_uuid}, 
  OwnerPubKey:  person.OwnerPubkey,
  Name:  ${name},
  Description: ${description}
  Tags: []string{"tag3", "tag4"}
  AppUrl: ${AppURl}
  }
  db.TestDb.CreateOrEditTribe(tribe)

• Get the total tribes to count using the DB function db.TestDb.GetTribesTotal()
• Assert that all the total tribes returned from the API response is equal to the response from the database

Assertions

• [ ] Assert that all the test passes

Acceptance Criteria

• [ ] Do not delete any of the existing test cases on the TestGetTotalTribes
• [ ] All test cases after the TestGetTotalTribes refactor passes
• [ ] The Refactoring of TestGetTotalTribes should not break existing test flows.
• [ ] I have rebased and tested locally before submitting my PR
• [ ] I can submit a PR within 1 day of taking the bounty

Here is an example Real DB Test

[MahtabBukhari]

@elraphty Could you please assign me?

[MuhammadUmer44]

@elraphty Please assign me?

[AbdulWahab3181]

@elraphty Could you please assign me?