Postgres Tests | Permify | Page 1

glass wagon Dec 6, 2023, 10:08 AM

#

Hello @pliant thorn , we have recently switched to Ginkgo for database tests, and we haven't labeled it, and it's working entirely according to Postgres specification. I will open an issue for this. Is this urgently needed for you?

pliant thorn Dec 6, 2023, 10:09 AM

#

glass wagon Hello <@309308003847503872> , we have recently switched to Ginkgo for database t...

Well I needed to test Data Bundle for the in memory db I was working on

#

But I get the same error

glass wagon Dec 6, 2023, 2:49 PM

#

Can you share the error there? If you write 'go test' inside internal/storage/postgres, it should work.

pliant thorn Dec 6, 2023, 7:55 PM

#

#

doing go test <filename> returns this like it did with the other tests

glass wagon Dec 7, 2023, 2:10 PM

#

Does it work when you write just go test in the path you are in?

pliant thorn Dec 9, 2023, 6:46 PM

#

Sorry about that was missing a test Spec function, got the tests passing now

#

though I'm unable to fix this import cycle not allowed error when trying to use a database factory, I've also tried just making a memdb instance directly but importing schema gets me back to that same error

#

Currently for the tests the schema is hardcoded in the test itself

#

@glass wagon Could you look at this if you have time?

pliant thorn Dec 11, 2023, 12:36 PM

#

@glass wagon Hi sorry to tag you again, just bumping this in case you may missed it.

glass wagon Dec 11, 2023, 1:26 PM

#

Hello @pliant thorn , can you try creating a database without using a factory? You can refer to the Postgres test example. Or, you can copy the code inside the factory and run it here.

pliant thorn Dec 11, 2023, 1:28 PM

#

glass wagon Hello <@309308003847503872> , can you try creating a database without using a fa...

in the 2nd screenshot I tried making the memdb instance directory without factory, but importing migrations schema caused import cycle again

#

I’m quite new to Go lang coming from Typescript so this is a bit confusing

glass wagon Dec 11, 2023, 1:30 PM

#

Can you send me the list of cycles? It should appear in the details of this error message.

pliant thorn Dec 11, 2023, 1:33 PM

#

glass wagon Can you send me the list of cycles? It should appear in the details of this erro...

Doesn’t shows me a list that’s the entire error when I hover over it

glass wagon Dec 11, 2023, 1:48 PM

#

Can you try running it in the terminal? Full error will be visible in the terminal

pliant thorn Dec 11, 2023, 2:05 PM

#

glass wagon Can you try running it in the terminal? Full error will be visible in the termin...

pliant thorn Dec 11, 2023, 2:29 PM

#

I figured it out, moved the table name consts to a new package that did the trick, cleaning it up for the PR now

pliant thorn Dec 14, 2023, 2:40 PM

#

@placid tree Hi, I was wondering if a partial bounty for DataBundles issue is possible? I've got Bundle Read, Write & Delete to work, but haven't been able to make RunBundle work

#

The tests are passing for Read, Write & Delete

#

placid tree Dec 14, 2023, 2:45 PM

#

That can be doable, but since I don’t know the exact technical details lets get @glass wagon’s thoughts on this.

#Postgres Tests