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

[Bugfix] Fixes For Failing Tests #1489

Draft
wants to merge 63 commits into
base: develop
Choose a base branch
from

Conversation

Civolilah
Copy link
Collaborator

@Civolilah Civolilah commented Jan 25, 2024

@beganovich @turbo124 After finishing the permissions adjustment, 98% of tests are fixed and working great. This PR addresses only a few tests that are not covered by the permissions adjustment. Currently, locally, when I run all tests after seeding the initial database, all of them pass, more than 190 in total. However, on GitHub Actions, all of them fail. I believe this is due to a slight environmental adjustment needed there.

From the current status of tests, it appears that the login functionality in the app is unavailable. None of the tests are even executing their intended actions, they all fail at the login stage.

Therefore, as I see the situation now, I believe that fixing the connection with the API and DB will resolve this issue. Additionally, we need to ensure that the VITE_ROUTER environment variable is set to "browser" but NOT "hash," and VITE_IS_TEST variable is set to "true".

If I can assist in making any of these fixes, please let me know.

@Civolilah Civolilah marked this pull request as ready for review January 26, 2024 16:51
@beganovich
Copy link
Member

I guess it should be a good idea to fix GitHub actions in this PR? Is that something you can have a look at at some point @turbo124?

@turbo124
Copy link
Member

turbo124 commented Feb 5, 2024

I guess it should be a good idea to fix GitHub actions in this PR? Is that something you can have a look at at some point @turbo124?

I'm seeing green currently?

@Civolilah Civolilah marked this pull request as draft February 5, 2024 18:44
@Civolilah
Copy link
Collaborator Author

@beganovich @turbo124 David made a PR today to fix actions, added some .env variables, and set up some other small things, but for some reason it still does not work. Writing a comment just to be aligned. Once I'm done with some other tasks, I can take a look at the whole workflow and try to replicate why it does not work the same as locally if you do not resolve it before me.

@beganovich
Copy link
Member

@Civolilah is this still draft?

@Civolilah
Copy link
Collaborator Author

@Civolilah is this still draft?

@beganovich Yes, David and I worked on fixing GitHub actions to ensure the tests work the same as locally. We encountered a small problem with the API port, but David managed to find a solution. Currently, it is up to me to identify the reasons for some other minor bugs, and once I resolve them and pass the GitHub actions tests as we do locally, it will be ready for review.

We had some more important tasks than this one, so I addressed it once those were completed. However, I will prioritize fixing this asap and will keep you informed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants