Briheet Singh Yadav
Briheet Singh Yadav
**Describe the bug** While checking out the issue with improving repo test coverage, i ran `mage tests` which gave the following results  `groupjobs_test.go:1113: Error Trace: /home/briheet/Open/armada/internal/lookoutv2/repository/groupjobs_test.go:1113 /home/briheet/Open/armada/internal/lookoutv2/repository/groupjobs_test.go:28 /home/briheet/Open/armada/internal/common/database/db_testutil.go:59 /home/briheet/Open/armada/internal/common/database/lookout/util.go:15...
both NewSession and NewSessionNonBlocking use NewSessionCommon which has this https://github.com/scylladb/gocql/blob/5df796945891c67fa322c23f751e9d885abf5842/session.go#L129 We could have something as NewSessionWithContext take in a context which we pass and default context.Background() for previous 2 sessions...