Projet

Général

Profil

Bug #42634

test_convert_to_sql.py: conflit sur le nom de base

Ajouté par Thomas Noël il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ca peut arriver, ça arrive :

    @pytest.fixture
    def database(cursor):
        dbname = 'wcstests%d' % random.randint(0, 100000)
>       cursor.execute('CREATE DATABASE %s' % dbname)
E       psycopg2.ProgrammingError: ERREUR:  la base de donn\xe9es \xab wcstests3202 \xbb existe d\xe9j\xe0

Fichiers

Révisions associées

Révision 7e07d08a (diff)
Ajouté par Thomas Noël il y a presque 4 ans

tests: try multiple times to get a database name (#42634)

Historique

#1

Mis à jour par Thomas Noël il y a presque 4 ans

#2

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

J'aurais dit plutôt essayer de vérifier si elle existe avant de la créer, auquel cas la shooter (avant de la re-créer donc), et si ça merde encore lâcher l'affaire.

#3

Mis à jour par Frédéric Péters il y a presque 4 ans

Ça peut être une base d'une exécution d'un job parallèle, faut pas toucher.

#4

Mis à jour par Thomas Noël il y a presque 4 ans

Frédéric Péters a écrit :

Ça peut être une base d'une exécution d'un job parallèle, faut pas toucher.

C'est ce que je fais. En fait mon patch ne fait que reprendre la technique de sioux présente dans tests/utilities.py pour la création des bases principales. On avait juste oublié qu'on faisait aussi des bases lors du test de conversion, d'où ma proposition copier-coller-bête-et-méchant.

#5

Mis à jour par Frédéric Péters il y a presque 4 ans

(oui, je répondais à Manu).

#6

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

Si vous avez vos techniques de sioux aussi, je peux pas suivre, moi apache.

(je laisse la validation aux sioux).

#7

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée
#8

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 7e07d08a679fbbda6619414e00a32e69020bd62e
Author: Thomas Noël <tnoel@entrouvert.com>
Date:   Thu May 7 17:22:56 2020 +0200

    tests: try multiple times to get a database name (#42634)
#9

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF