Ensure namespace list option is propagated to the descriptors client lister

[danail-branekov]

Steps to reproduce:

• Change this test to

		It("returns a not found error", func() {
			Expect(err).To(BeAssignableToTypeOf(apierrors.NotFoundError{}))
		})

• Run the test, see it fails

Dev notes

The reposotiry performs a get via list. List should return an empty result in case the user is not authorised in the corresponding namespace.

When the repostiory lists the builds, it specifies the InNamespace option, however, that namespace option is not propagated when performing the list with the rest client

What we need to do is to make sure that this option is used when performing the rest client list.

[souravpy]

Can I be assigned this issue? Ill be glad to work

[danail-branekov]

Can I be assigned this issue? Ill be glad to work

Sure, feel free to pick it up. Looking forward the PR

[souravpy]

Hello Danail. I made the suggested changes to the test file and running make test.
It returns the following

•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••

Summarizing 1 Failure:
[FAIL] BuildRepository GetLatestBuildByAppGUID [It] returns a not found error
/home/sourav/korifi/api/repositories/build_repository_test.go:200

Ran 613 of 613 Specs in 218.276 seconds
FAIL! -- 612 Passed | 1 Failed | 0 Pending | 0 Skipped

[1764849258] Authorization Suite - 62/62 specs - 4 procs •••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••• SUCCESS! 37.51613907s
[1764849258] Parse Suite - 5/5 specs - 4 procs ••••• SUCCESS! 54.289067ms
[1764849258] Manifest Suite - 82/82 specs - 4 procs •••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••• SUCCESS! 640.87427ms
[1764849258] Payloads Suite - 643/643 specs - 4 procs ••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••â�¢•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••• SUCCESS! 1.707908044s
[1764849258] Stats Suite - 11/11 specs - 4 procs •••â�¢••• SUCCESS! 113.991718ms

Ginkgo ran 24 suites in 15m20.383568581s

There were failures detected in the following suites:
repositories ./repositories

Test Suite Failed
exit status 1
make[1]: *** [Makefile:44: test] Error 1
make[1]: Leaving directory '/home/sourav
/korifi/api'
make: *** [Makefile:59: test] Error 2

[danail-branekov]

Hi @souravpy

Having a failing test is a good starting point. Next, you need to fix it :)

In order to fix it, you could try to somehow scope the Get request here down to the namespace specified in the list options, or do not scope it if that option is not set.

I would recommend that you first start with a failing test within this decribe. The test could create apps in a couple of namespaces and the list request could have its client.InNamespace option set. Then it should only get apps from the specified namespace.
Of course, there should be another test that lists without the InNamespace option set - such a list should return all apps from all namespaces.