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

Increase test coverage backend #1828

Closed
6 tasks done
metemaddar opened this issue Jan 4, 2023 · 3 comments
Closed
6 tasks done

Increase test coverage backend #1828

metemaddar opened this issue Jan 4, 2023 · 3 comments
Milestone

Comments

@metemaddar
Copy link
Contributor

metemaddar commented Jan 4, 2023

Goal of this issue

Add the tests which are mentioned.
These tests are mentioned by TODO inside tests directory.

Resources

List of TODOs:

Tasks

Preview Give feedback
  1. API Testing
    metemaddar
  2. API Testing
    metemaddar
  3. metemaddar
  4. majkshkurti
  5. API Testing
    metemaddar

Deliverables

Branch to derive

dev

@majkshkurti majkshkurti added this to the v1.5 milestone Jan 5, 2023
@EPajares EPajares changed the title Add absent tests which are mentioned in tests files Increase test coverage backend Apr 11, 2023
@metemaddar
Copy link
Contributor Author

Moved from issue #1946

  • As Majk Suggests, we can use the Celery test doc for testing without workers running.
  • For Heatmaps as we need the Cache data, Majk suggests to run data preparation test before testing heatmaps. So we will have the heatmap data.
  • For testing scheduled endpoints, we need to avoid running them at the same time, as the endpoint may be busy with another long time taking task.
  • But as we are testing against python endpoint, we have the boilerplate at endpoint, we can reduce the vs-code tester workers to 1
  • We also need to set this at runners configurations.

@metemaddar
Copy link
Contributor Author

The Tests are now sequential by 6878c2d

I'm trying to make tests independent from REDIS (Results backend) if possible.

@metemaddar
Copy link
Contributor Author

metemaddar commented Apr 17, 2023

The indicators async implementation was done 8c6f732. And there were a concern about the client for if the async/await of JavaScript still makes CPU busy and hot. Fortunately as ChatGPT says, it will not block the CPU. So we can implement the indicators without set/result endpoints, and at the client side we can get the results asynchronously on one request.

From Chat GPT:

The async/await syntax in JavaScript is used to write asynchronous code that runs concurrently without blocking the execution of other tasks. It does not necessarily make the CPU busy and hot, but it depends on the implementation of the asynchronous tasks and how they are scheduled.

In general, asynchronous tasks such as network I/O or disk I/O operations are non-blocking and do not consume CPU resources while waiting for the operation to complete. Instead, the operating system or runtime environment handles the task and signals the application when the operation is complete.

@EPajares EPajares modified the milestones: v1.5, v1.6 Jun 6, 2023
@majkshkurti majkshkurti removed the Epic label Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants