Projet

Général

Profil

Bug #62192

base postgresql à créer pour que les tests wcs puissent être exécutés (?) (AttributeError: 'NoneType' object has no attribute 'cursor')

Ajouté par Corentin Séchet il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
25 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

    def get_connection_and_cursor(new=False):
        conn = get_connection(new=new)
        try:
>           cur = conn.cursor()
E           AttributeError: 'NoneType' object has no attribute 'cursor'

wcs/sql.py:482: AttributeError

Est survenu la première fois que j'ai lancé les tests sur WCS, le problème est résolu si je crée une table postgresql "csechet" à la main.


Fichiers


Demandes liées

Lié à BiJoe - Development #68312: Tests qui crashent en local en cherchant une base au nom de l'utilisateur linuxFermé22 août 2022

Actions

Révisions associées

Révision 61108724 (diff)
Ajouté par Emmanuel Cazenave il y a environ 2 ans

tests: do not rely on user database (#62192)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Description mis à jour (diff)
#2

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Projet changé de w.c.s. à Publik Installation Développeur

Comment lances-tu les tests ?

J'envoie vers publik-devinst voir s'il y manque quelque chose dans l'initialisation postgresql.

#3

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Sujet changé de AttributeError: 'NoneType' object has no attribute 'cursor' à base postgresql à créer pour que les tests wcs puissent être exécutés (?) (AttributeError: 'NoneType' object has no attribute 'cursor')
#4

Mis à jour par Corentin Séchet il y a environ 2 ans

J'ai lancé juste en faisant "tox" dans le répertoire de WCS.

#5

Mis à jour par Nicolas Roche il y a environ 2 ans

J'avais noté la même erreur en arrivant (il y a 3 ans)

   E       OperationalError: FATAL:  la base de données « nroche » n'existe pas

#6

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

https://www.postgresql.org/docs/current/libpq-connect.html#LIBPQ-PARAMKEYWORDS :

dbname
The database name. Defaults to be the same as the user name

#7

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

  • Projet changé de Publik Installation Développeur à w.c.s.
#8

Mis à jour par Frédéric Péters il y a environ 2 ans

Mais ça va juste râler en disant, FATAL: le rôle « fred » n'existe pas, non ? Et laisser de toute façon une configuration à faire en local, et c'est là dans les instructions qu'il manquerait un createdb fred ?

#9

Mis à jour par Frédéric Péters il y a environ 2 ans

Je me réponds, il y a déjà :

- name: "create the postgresql user named {{user}}" 
  postgresql_user:
    name: "{{user}}" 
    role_attr_flags: SUPERUSER,LOGIN
    port: "{{postgresql_port}}" 

  become: yes
  become_user: postgres

(et ça aurait pu continuer là-dedans, j'imagine, mais ok)

#10

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

  • Projet changé de w.c.s. à Publik Installation Développeur
  • Statut changé de Solution proposée à En cours

A j'avais oublié ça, ok continuons dans devinst si tu veux, I don't care.

#12

Mis à jour par Frédéric Péters il y a environ 2 ans

Je disais "mais ok" et vraiment tu avais fait le patch ça m'allait tout à fait pour wcs, c'était juste qu'à la lecture je m'interrogeais sur le bout que je ne voyais pas.

ici, "create the postgresql databse named" message à revoir, s/databse/user/

#13

Mis à jour par Frédéric Péters il y a environ 2 ans

Et ça échoue dans jenkins, t'ennuie pas et pousse le patch wcs, je dirais. (si quelqu'un pouvait le relire/tester formellement).

#14

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

  • Projet changé de Publik Installation Développeur à w.c.s.
  • Statut changé de Solution proposée à Résolu (à déployer)

Ça passe dans jenkins, et j'ai bien testé que ça corrigeait le problème en local, je pousse.

commit 611087242856bd2018b416cc6e64a2982e21f4bc
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Apr 6 17:20:07 2022 +0200

    tests: do not rely on user database (#62192)
#15

Mis à jour par Transition automatique il y a environ 2 ans

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

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

#17

Mis à jour par A. Berriot il y a plus d'un an

  • Lié à Development #68312: Tests qui crashent en local en cherchant une base au nom de l'utilisateur linux ajouté

Formats disponibles : Atom PDF