Resolve "Flying team location management site"
All threads resolved!
All threads resolved!
Closes #178 (closed)
Merge request reports
Activity
Filter activity
assigned to @jacek.lebioda
I populated unit tests for you. There are few changes that I introduced regarding it:
- your idea was almost ok :). The difference is that forma data is not based on inputs id, but name, therefore your form dist should be modified a bit. Take a look here https://git-r3lab.uni.lu/NCER-PD/scheduling-system/merge_requests/118/diffs#d3856ab8910b83d42b81f0e10850d25899a2c499_0_32
- Second diffference is that after saving form you should get 302 not 200 (there is redirection to flying team list on success)
- I added basic rendering test for editing flying team
- I change signature of
generate_more_or_less_random_nameinto static
1 3 from django.urls import reverse 2 4 5 from web.tests.functions import create_flying_team 3 6 from web.models import FlyingTeam 4 7 from web.tests import LoggedInTestCase 5 8 9 logger = logging.getLogger(__name__) 10 6 11 7 12 class FlyingTeamTests(LoggedInTestCase): 8 def generate_more_or_less_random_name(self): 13 @staticmethod 14 def generate_more_or_less_random_name(): 9 15 import random 10 letters = [chr(x) for x in range(97,122)] 11 return 'Radom' + ''.join(random.choice(letters) for x in range(15)) added 23 commits
-
d9e2af58...143dba58 - 22 commits from branch
master - ab50b6b8 - Merge branch 'master' into '178-flying-team-management-page'
-
d9e2af58...143dba58 - 22 commits from branch
enabled an automatic merge when the pipeline for ab50b6b8 succeeds
mentioned in commit e4ab6d2e
Great, I'll follow with #128 (closed) then!
Please register or sign in to reply