Project

General

Profile

Development #50075

tests: nom de base de données trop long

Added by Thomas Noël 14 days ago. Updated 7 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
12 Jan 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

malgré le [:63] dans tests/settings.py on peut avoir :

E           django.core.exceptions.ImproperlyConfigured: The database name 'test_combo-test-wip-50067-lingo-can-pay-only-once-with-remote-regie' (67 characters) is longer than PostgreSQL's limit of 63 characters. Supply a shorter NAME in settings.DATABASES.

parce qu'il y a un test_ ajouté (je ne sais pas d'où il vient, mais il est là).


Files

Associated revisions

Revision 37eebfff (diff)
Added by Thomas Noël 14 days ago

tests: shorten database name (#50075)

Revision 38c7ec1b (diff)
Added by Thomas Noël 14 days ago

tests: shorten database name, again (#50075)

History

#1

Updated by Thomas Noël 14 days ago

On passe de

        'NAME': 'combo-test-%s' % os.environ.get("BRANCH_NAME", "").replace('/', '-')[:63]

à
        'NAME': 'combo-test-%s' % os.environ.get("BRANCH_NAME", "").replace('/', '-')[:55]

parce qu'en plus le :63 ne s'appliquait qu'au nom de la branche, alors que va y être ajouté "test_combo-test-" donc on coupe à 55.

#2

Updated by Thomas Noël 14 days ago

(testé sur jenkins via #50067)

#3

Updated by Emmanuel Cazenave 14 days ago

  • Status changed from Solution proposée to Solution validée
#4

Updated by Thomas Noël 14 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 37eebfff08294d5d435f0ecf492d57df87b05847
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Jan 12 14:04:22 2021 +0100

    tests: shorten database name (#50075)

#5

Updated by Thomas Noël 14 days ago

et quel imbécile, erreur de calcul, c'était :45 ... je pousse la triviale correction.

commit 38c7ec1bd606d019a4add99e8e9472f45594dba3
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Jan 12 14:28:59 2021 +0100

    tests: shorten database name, again (#50075)

#6

Updated by Frédéric Péters 7 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF