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

False unnecessary semi-colon report #255

Closed
westonal opened this issue Jul 20, 2018 · 2 comments
Closed

False unnecessary semi-colon report #255

westonal opened this issue Jul 20, 2018 · 2 comments

Comments

@westonal
Copy link

westonal commented Jul 20, 2018

Version 0.24.0

When using should throw the Exception or should throw from kluent you often end up doing this:

    @Test
    fun `kluent issue`() {
        val any = Any(); //some setup
        {
            any.toString()
        } `should throw` Exception::class
    }

The semi-colon is reported as unnecessary: Unnecessary semicolon (cannot be auto-corrected)

But Android Studio doesn't mark it as unnecessary:

image

and removing it causes:

image

@shyiko shyiko closed this as completed in d028b95 Jul 21, 2018
@shyiko
Copy link
Collaborator

shyiko commented Jul 21, 2018

Fix is coming in 0.25.0 (already in master).

@shyiko
Copy link
Collaborator

shyiko commented Jul 25, 2018

0.25.0 released.

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

No branches or pull requests

2 participants