Netz39_Vorstand/entities_validation_svc
3
0
Fork 0
Code Issues Pull requests 1 Releases 1 Activity

validation endpoint implementation #7

Merged
tux merged 19 commits from experiments-cerberus into master 2021-01-05 14:34:02 +01:00
Conversation 6 Commits 19 Files changed 9 +363 -5
dkdent commented 2020-12-03 14:31:24 +01:00
Collaborator
Copy link
No description provided.
tux was assigned by dkdent 2020-12-03 14:31:24 +01:00
dkdent commented 2020-12-04 21:19:22 +01:00
Author
Collaborator
Copy link

So, jetzt sieht man hier schon mal was.

So, jetzt sieht man hier schon mal was.
🎉 1
dkdent changed title from WIP: experiments-cerberus to experiments-cerberus 2020-12-16 07:18:43 +01:00
tux requested changes 2021-01-04 21:55:16 +01:00
tux left a comment
Owner
Copy link

Abgesehen von der einen Anmerkung finde ich das gut. :)

Abgesehen von der einen Anmerkung finde ich das gut. :)
test.py Outdated
@ -58,2 +72,4 @@
self.assertTrue(validation_result['valid'], "Validation result is expected to be valid==true")
def test_invalid_iban(self):
entity_file = json.load(open('test_cases/invalid/invalid_iban.json','r'))
tux commented 2021-01-04 21:54:58 +01:00
Owner
Copy link

Ich bekomme hier:

/home/tux/Netz39/Vorstand/entities_validation_svc/test.py:75: ResourceWarning: unclosed file <_io.TextIOWrapper name='test_cases/invalid/invalid_iban.json' mode='r' encoding='UTF-8'>
  entity_file = json.load(open('test_cases/invalid/invalid_iban.json','r'))
ResourceWarning: Enable tracemalloc to get the object allocation traceback

Besser wäre folgendes:

        with open('test_cases/invalid/invalid_iban.json', 'r') as f:
            entity_file = json.load(f)

Analog an den anderen Stellen.

Ich bekomme hier: ``` /home/tux/Netz39/Vorstand/entities_validation_svc/test.py:75: ResourceWarning: unclosed file <_io.TextIOWrapper name='test_cases/invalid/invalid_iban.json' mode='r' encoding='UTF-8'> entity_file = json.load(open('test_cases/invalid/invalid_iban.json','r')) ResourceWarning: Enable tracemalloc to get the object allocation traceback ``` Besser wäre folgendes: ```python with open('test_cases/invalid/invalid_iban.json', 'r') as f: entity_file = json.load(f) ``` Analog an den anderen Stellen.
tux commented 2021-01-04 21:56:10 +01:00
Owner
Copy link

Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben.

Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben.
dkdent commented 2021-01-04 22:13:37 +01:00
Author
Collaborator
Copy link

Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben.

Ich bin unsicher, ob das mit den 80 Zeichen Zeilen wirklich notwendig ist...
Das sieht teilweise ganz schön blöd aus.

> Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben. Ich bin unsicher, ob das mit den 80 Zeichen Zeilen wirklich notwendig ist... Das sieht teilweise ganz schön blöd aus.
tux commented 2021-01-04 22:29:53 +01:00
Owner
Copy link

Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben.

Ich bin unsicher, ob das mit den 80 Zeichen Zeilen wirklich notwendig ist...
Das sieht teilweise ganz schön blöd aus.

80 Zeilen fordert der Linter bei mir auch gar nicht - aber einiges in Bezug auf Einrückung und Leerzeilen.

> > Der PEP-Linter findet noch einiges - mit pycharm lässt sich das leicht finden und beheben. > > Ich bin unsicher, ob das mit den 80 Zeichen Zeilen wirklich notwendig ist... > Das sieht teilweise ganz schön blöd aus. 80 Zeilen fordert der Linter bei mir auch gar nicht - aber einiges in Bezug auf Einrückung und Leerzeilen.
dkdent changed title from experiments-cerberus to validation endpoint implementation 2021-01-04 23:16:35 +01:00
dkdent commented 2021-01-04 23:16:40 +01:00
Author
Collaborator
Copy link

done

done
👍 1
tux commented 2021-01-05 14:33:11 +01:00
Owner
Copy link

Du wolltest mit den Syntax-Fehlern in den Testcases das Review testen, oder? ;)

Du wolltest mit den Syntax-Fehlern in den Testcases das Review testen, oder? ;)
tux approved these changes 2021-01-05 14:33:35 +01:00
tux closed this pull request 2021-01-05 14:34:02 +01:00
tux deleted branch experiments-cerberus 2021-01-05 14:34:09 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
tux
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Netz39_Vorstand/entities_validation_svc#7
No description provided.
Delete branch "experiments-cerberus"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?