Skip to content

WrappingParameterizedRunner debug code causes failure on Windows #12

Description

@Nava2

This debug code:

// WrappingParameterizedRunner.java L313
/* ** DEBUG export class to /tmp/x.class ** */
		try {
			FileOutputStream fos = new FileOutputStream("/tmp/x.class");
			fos.write(clazzBytes);
			fos.close();
		} catch (Exception e1) {
			e1.printStackTrace();
		}
		/* ** /DEBUG ** */

throws on Windows immediately, /tmp doesn't exist. Could be replaced with the Files.createTempFile() or similar.

Activity

  1. xkr47 commented on Mar 27, 2017

    @xkr47
    Member

    *blush* It should just be removed. It was used during development.

  2. xkr47 commented on Mar 27, 2017

    @xkr47
    Member

    Actually it was fixed a year ago but no new release has been made. Maybe time for one then!

  3. Nava2 commented on Mar 27, 2017

    @Nava2
    Author

    I had a feeling.. Once JUnit 5 stabilizes their parameterized testing, hopefully it means this library isn't needed anymore and no more maintenance. 🙌

  4. IRasmivan commented on Jul 3, 2017

    @IRasmivan

    Any Plan to fix this!?

  5. xanyook commented on Jun 11, 2018

    @xanyook

    hello,

    any chance for a release on this correction (fe4d2d9) (thx xkr47) ?
    this lib looks nice but can't run it on local due to this issue.

    Had to fix it myself bu overriding WrappingParameterizedRunner but need to add ParameterizedSuiteBuilder, DescriptionMappingRunNotifierProxy, ClassLoaderInjector sources too, it s not clean in my project :(

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

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions