Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

Initial integration tests #3483

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

glanham-jr
Copy link
Contributor

@glanham-jr glanham-jr commented Mar 9, 2023

What does this PR do?

Creates a base integration test suite to quickly test search results of a set of listed engines.

This is a PR in progress and is meant to aid in diagnosing certain engine issues.

Why is this change important?

Allows for quick iteration of engine development for selected engines (more could be added).

How to test this PR locally?

Run the following command in your terminal

make test.int

image

Author's checklist

Related issues

@glanham-jr glanham-jr marked this pull request as draft March 9, 2023 02:59
@kvch
Copy link
Member

kvch commented Apr 4, 2023

Is this a similar checker to searx-checker? https://github.com/searx/searx-checker

@glanham-jr
Copy link
Contributor Author

glanham-jr commented Apr 4, 2023

Yes it is, didn't know that existed. While these integrations tests don't have to be ran for CI/CD, it provides a way to quickly test engines locally at the run of a command.

Some things I want to do:

  1. Pass in what engine(s) you want to run on the command line.
  2. Only a subset of engines are supported, specifically those that require additional setup. Haven't planned how to structure that, but was thinking to pull setting from a config file.

With this, if the integration tests are stable, it might be nice to include them in CI for the most common engines (even if not failing the build) so we are alerted on engine issues much faster.

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

Successfully merging this pull request may close these issues.

None yet

2 participants