Skip to content

Pov exercise tests seem to have missing commas #445

Description

@hawaka6667-dev

I was working on the PowerShell pov exercise, but all 16 tests fail before the tests seem to actually run.

I found what looks like a syntax issue in Pov.Tests.ps1.

For example, this:

[Tree]::new("x", @([Tree]::new("kid-0"), [Tree]::new("kid-1")))
[Tree]::new("sibling-0")
[Tree]::new("sibling-1")

seems to need commas between the array elements:

[Tree]::new("x", @(
    [Tree]::new("kid-0"),
    [Tree]::new("kid-1")
)),
[Tree]::new("sibling-0"),
[Tree]::new("sibling-1")

There is a similar issue in the Can reroot a rerooted tree back to the original tree test.

Because of this, Pester reports the tests as failing during setup rather than giving normal assertion failures.

Could someone confirm whether these commas are missing from the current test file?

If this is indeed a bug, would you be open to a PR fixing the test file? I'd be happy to submit the fix.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions