Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Java piscine / BreakdownURL subject #2830

Open
liogedz opened this issue Dec 19, 2024 · 2 comments
Open

[BUG] Java piscine / BreakdownURL subject #2830

liogedz opened this issue Dec 19, 2024 · 2 comments
Assignees
Labels

Comments

@liogedz
Copy link

liogedz commented Dec 19, 2024

Describe the bug

Tester is comparing the size of the returning map to be equal to 2 on the actual size of 3.

    @Test
    void testParseURL_withoutPortAndPath() {
        String url = "https://www.example.com";
        Map<String, String> components = parser.parseURL(url);
        assertThat(components)
                .withFailMessage("Expected components to be {protocol=https, domain=www.example.com, path=/} but got %s", components)
                .containsEntry("protocol", "https")
                .containsEntry("domain", "www.example.com")
                .hasSize(2);  // Ensure only these 3 components are present
    }

Users

Severity
Choose the severity label (⚠️ critical).

Type
Choose the label (⚙️ functionality)

To Reproduce
Steps to reproduce the behavior:

Run tester.

Workarounds
Any known workarounds for the issue:
Change number to 3 here:
.hasSize(2);

Expected behavior
A clear and concise description of what you expected to happen.
Change number to 3 here:
.hasSize(2);

Attachments
https://drive.google.com/file/d/1anlGZg83Fo29dQcJ-ET3-byDPvyoEgo_/view?usp=sharing

Desktop (please complete the following information):

Ubuntu 24.04.01 LTS

Smartphone (please complete the following information):
na

Additional context
subject link:
https://github.com/01-edu/public/tree/master/subjects/java/checkpoints/breakdown-url

@zanninso
Copy link
Contributor

Hello @liogedz thank for reporting that,
I'm on it the fix will be submitted as soon as possible

@liogedz
Copy link
Author

liogedz commented Dec 22, 2024

Hey!
Thank you for the prompt reply!
Managed to run those docker tests localy on Intellij - it works with 3 instead of 2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants