Skip to content

Add opening hours link - #2723

Open
cyrillkuettel wants to merge 1 commit into
masterfrom
ogc-3427-offnungszeiten-als-standardlink
Open

cyrillkuettel wants to merge 1 commit into
masterfrom
ogc-3427-offnungszeiten-als-standardlink

Conversation

@cyrillkuettel

Copy link
Copy Markdown
Contributor

Please fill in the commit message below and work through the checklist. You can delete parts that are not needed, e.g. the optional description, the link to a ticket or irrelevant options of the checklist.

Commit message

Town6: Add opening hours link

TYPE: Feature
LINK: OGC-3427

Checklist

  • I have performed a self-review of my code
  • I have tested my code thoroughly by hand
  • I have added tests for my changes/features

@linear

linear Bot commented Oct 1, 2026

Copy link
Copy Markdown

OGC-3427

@codecov

codecov Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
2964 1 2963 21
View the top 1 failed test(s) by shortest run time
tests/onegov/feriennet/test_browser.py::test_volunteers_export[open-Offen]
Stack Traces | 21.1s run time
browser = <tests.shared.browser.ExtendedBrowser object at 0x7f09648eeba0>
scenario = <tests.onegov.activity.conftest.Scenario object at 0x7f0965458440>
state = 'open', label = 'Offen'

    @pytest.mark.parametrize('state,label', [
        ('open', 'Offen'),
        ('contacted', 'Kontaktiert'),
        ('confirmed', 'Bestätigt'),
        ('cancelled', 'Abgelehnt'),
    ])
    def test_volunteers_export(
        browser: ExtendedBrowser,
        scenario: Scenario,
        state: str,
        label: str
    ) -> None:
    
        scenario.add_period(title="Ferienpass 2019", active=True, confirmed=True)
        scenario.add_activity(title="Zoo", state='accepted')
        scenario.add_occasion(age=(0, 10), spots=(0, 2), cost=100)
        scenario.add_need(
            name="Begleiter",
            number=BoundedIntegerRange(1, 4),
            accept_signups=True
        )
        scenario.add_volunteer(
            state=state,
            first_name="Foo",
            last_name="Bar",
            birth_date=date(1984, 6, 4),
            organisation="",
            address="Foostreet 1",
            zip_code="1234",
            place="Bartown",
            email="foo@bar.org",
            phone="1234"
        )
        scenario.commit()
        scenario.refresh()
    
        page = browser.page
>       browser.login_admin()

.../onegov/feriennet/test_browser.py:303: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
tests/shared/browser.py:364: in login_admin
    self.login('admin@example.org', 'hunter2', to)
tests/shared/browser.py:358: in login
    self.visit(url)
tests/shared/browser.py:349: in visit
    self.fail_on_console_errors(sleep_before_fail, expected_errors)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <tests.shared.browser.ExtendedBrowser object at 0x7f09648eeba0>
sleep_before = 0, expected_errors = []

    def fail_on_console_errors(
        self,
        sleep_before: float = 0,
        expected_errors: list[dict[str, Any]] | None = None
    ) -> None:
        expected_errors = expected_errors or []
        filters = expected_errors + self.failsafe_filters
        error_msgs = self.get_console_log(filters)
        if error_msgs and environ.get('SHOW_BROWSER') == '1':
            time.sleep(sleep_before)
>       assert not error_msgs, error_msgs
E       AssertionError: [{'level': 'SEVERE', 'source': 'console-api', 'message': 'http://127.0.0.1:39223/auth/login - Failed to load resource: the server responded with a status of 500 (INTERNAL SERVER ERROR)'}]
E       assert not [{'level': 'SEVERE', 'source': 'console-api', 'message': 'http://127.0.0.1:39223/auth/login - Failed to load resource: the server responded with a status of 500 (INTERNAL SERVER ERROR)'}]

tests/shared/browser.py:672: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant