Implementation of the sequential test mechanism - [merged] #120

Closed benoitdm-oslandia closed 1 year ago

In GitLab by @lowzonenose on Oct 12, 2022, 17:05

_Merges feature/sequentialtests -> develop

Ref #48 : Implémentation du mécanisme de tests séquentiels (ordre d’exécution) pour le mode db et mock

Structure d'un fichier Runner :

package db_test

type DbTests struct {
  Test *testing.T
}

func TestMain(m *testing.M) {
  // some stuff...
}

func TestRunnerHandlerDb(t *testing.T) {

  // initialisation avant l'execution des tests
  beforeRun()

  // liste de tests sur la suppression des features
  t.Run("DELETE", func(t *testing.T) {
    beforeEachRun()
    test := DbTests{Test: t}
    test.TestDeleteFeatureDb()
    afterEachRun()
  })

  // nettoyage après execution des tests
  afterRun()
}

Ex. d'une fonction de test :

func (t *DbTests) TestDeleteFeatureDb() {
  // some stuff...
}

Autre ex. de fonction de test :

func (t *DbTests) TestDeleteFeatureDb() {
  t.Test.Run("TestDeleteFeatureDb", func(t *testing.T) {
    // some stuff...
  })
}

Il est toujours possible d'ajouter des tests classiques sans passer par un runner.

Comment lancer les tests ?

tous les mocks

$ go test -run ^TestRunnerHandlerMock$ github.com/CrunchyData/pg_featureserv/internal/service/mock_test

un groupe

$ go test -run ^TestRunnerHandlerMock/DELETE$ github.com/CrunchyData/pg_featureserv/internal/service/mock_test

un test

$ go test -run ^TestRunnerHandlerMock/DELETE/TestDeleteExistingFeature$ github.com/CrunchyData/pg_featureserv/internal/service/mock_test

In GitLab by @lowzonenose on Oct 12, 2022, 17:11

added 1 commit

2496102d - Fix linter

Compare with previous version

In GitLab by @lowzonenose on Oct 12, 2022, 17:18

@benoitdm-oslandia pourquoi le test "TestProperDbInit" sur la CI nous dit qu'il y'a 5 tables alors qu'il ne peut en avoir que 4 (mock_a, mock_b, mock_c et mock_multi) !?

In GitLab by @lowzonenose on Oct 12, 2022, 22:52

added 4 commits

2496102d...1d4ebd4c - 2 commits from branch develop
859baad4 - Implementation of the sequential test mechanism (db mode and mock)
164e0977 - Fix linter

Compare with previous version

In GitLab by @lowzonenose on Oct 12, 2022, 22:53

requested review from @benoitdm-oslandia

In GitLab by @lowzonenose on Oct 12, 2022, 22:58

added 1 commit

8a7e01df - Fix on the number of tables in the database (CI)

Compare with previous version

In GitLab by @lowzonenose on Oct 12, 2022, 23:43

added 1 commit

62c47e55 - Fix tests GET html (path assets)

Compare with previous version

In GitLab by @lowzonenose on Oct 13, 2022, 12:58

added 3 commits

ed04842d - 1 commit from branch develop
89e67ed6 - Implementation of the sequential test mechanism (db mode and mock)
6d0df94d - Fix linter

Compare with previous version

In GitLab by @lowzonenose on Oct 13, 2022, 13:31

added 1 commit

c1b1ce73 - Clean Db before each test

Compare with previous version

parce que il lit une table système qui n'est pas exclue ("tiger.tabblock20") mais qui le devrait

Avant de faire la review ton explication dans le corps de ticket mériterait de partir dans un fichier de doc hugo ! Nouvelle tâche pour toi :smile:

comment cette indentation pourrie a pu passer les checks ??

pour le coup j'en ai aussi 5 en local

je voulais le faire mais j'avais la flemme :smile:

pourquoi ?

Good job !

Si le changement de chemin de assets est lié au déplacement des tests alors il faut trouver une solution qui prenne trouve les assets quelque soit l'appelant.

Sinon top!

In GitLab by @lowzonenose on Oct 13, 2022, 16:12

hummm..., du coup, ce test est un peu pourri !?

In GitLab by @lowzonenose on Oct 13, 2022, 16:14

Commented on internal/service/db_test/handler_db_test.go line 34

pas terrible comme test si les tables sys ne sont pas écartées...

In GitLab by @lowzonenose on Oct 13, 2022, 16:14

Commented on internal/util/test_http.go line 54

changement de niveau des tests dans la structure de fichier...

In GitLab by @lowzonenose on Oct 13, 2022, 16:17

resolved all threads

oui on pourrait l'ajouter par défaut dans les tables exclue ou bien modifier le fichier de conf utilisée pour la spécifier précisément

In GitLab by @lowzonenose on Oct 13, 2022, 16:49

c'est une conf uniquement valable pour les tests..., a t on besoin de rendre le truc dynamique ?

In GitLab by @lowzonenose on Oct 13, 2022, 16:54

Commented on internal/service/db_test/handler_db_test.go line 34

changed this line in version 9 of the diff

In GitLab by @lowzonenose on Oct 13, 2022, 16:54

added 4 commits

b1318a1d - 1 commit from branch develop
c85f5740 - Implementation of the sequential test mechanism (db mode and mock)
856333f6 - Fix linter
1c004031 - Clean Db before each test

Compare with previous version

le truc ne me semblait pas complexe mais je réfléchissait en python où c assez simple à faire.

Est ce que tu peux jeter un coup d'oeil rapide stp ?

In GitLab by @lowzonenose on Oct 13, 2022, 17:15

Commented on internal/util/test_http.go line 54

changed this line in version 10 of the diff

In GitLab by @lowzonenose on Oct 13, 2022, 17:15

added 1 commit

5bafd8e9 - Fix assets into tests for resource html

Compare with previous version

In GitLab by @lowzonenose on Oct 13, 2022, 17:16

le truc rapide, c'est d'ajouter les assets en param.

cf. last commit

très bien

In GitLab by @lowzonenose on Oct 13, 2022, 17:25

resolved all threads

@lowzonenose est ce que tu peux ajouter ton explication dans le corps de ticket dans un fichier de doc hugo stp ?

In GitLab by @lowzonenose on Oct 13, 2022, 17:59

ok, je vais essayer de trouver un emplacement pour documenter les tests.

In GitLab by @lowzonenose on Oct 13, 2022, 21:25

added 2 commits

7eb40a71 - Fix make test command +
a64290eb - Add doc on how to run tests

Compare with previous version

In GitLab by @lowzonenose on Oct 13, 2022, 21:27

resolved all threads

In GitLab by @lowzonenose on Oct 13, 2022, 21:31

added 1 commit

b3c1625b - Fix linter on markdown file

Compare with previous version

    go test -v $(go list ./... | grep -v /vendor/)

  go test -v $(go list ./... | grep -v /vendor/)

Bon très bien, il faudra squasher une partie des commits (comme les 2 derniers) et puis c'est bon

In GitLab by @lowzonenose on Oct 14, 2022, 10:47

Commented on hugo/content/installation/installing.md line 61

cette commande va exécuter les tests sur la BDD.

Et, il y'a des conditions à leurs exécutions, il faut exposer DATABASE_URL.

soit on les exclue de l’exécution des tests (ex go list ./... | grep -vw db_test)
soit on teste que la variable d'environnement est positionnés

choix ?

ok ! Conditions qu'il faudra d'ailleurs documenter :/

In GitLab by @lowzonenose on Oct 14, 2022, 11:21

Commented on Makefile line 43

changed this line in version 13 of the diff

In GitLab by @lowzonenose on Oct 14, 2022, 11:21

added 1 commit

30479bd7 - Fix make test

Compare with previous version

In GitLab by @lowzonenose on Oct 14, 2022, 11:27

resolved all threads

In GitLab by @lowzonenose on Oct 14, 2022, 11:28

approved this merge request

j'ai eu un pbm de lecture ... il me manquait des bouts tout à l'heure :

je veix bien que tu remplaces par :

go test -v $(go list ./... | grep -vw db_test)

resolved all threads

benoitdm-oslandia / pg_featureserv

Implementation of the sequential test mechanism - [merged] #120