Polish "Ignore spock.lang annotations when creating test context cache key"

See gh-8252
pull/8458/head
Andy Wilkinson 8 years ago
parent 8f18df8a9c
commit 98cf35d48e

@ -1,5 +1,5 @@
/* /*
* Copyright 2012-2016 the original author or authors. * Copyright 2012-2017 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -324,8 +324,8 @@ class ImportsContextCustomizer implements ContextCustomizer {
@Override @Override
public boolean isIgnored(Annotation annotation) { public boolean isIgnored(Annotation annotation) {
return annotation.annotationType().getName().startsWith("org.spockframework.") || return annotation.annotationType().getName().startsWith("org.spockframework.")
annotation.annotationType().getName().startsWith("spock."); || annotation.annotationType().getName().startsWith("spock.");
} }
} }

@ -1,5 +1,5 @@
/* /*
* Copyright 2012-2016 the original author or authors. * Copyright 2012-2017 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -19,6 +19,8 @@ package org.springframework.boot.test.context;
import kotlin.Metadata; import kotlin.Metadata;
import org.junit.Test; import org.junit.Test;
import org.spockframework.runtime.model.SpecMetadata; import org.spockframework.runtime.model.SpecMetadata;
import spock.lang.Issue;
import spock.lang.Stepwise;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
@ -37,10 +39,18 @@ public class ImportsContextCustomizerTests {
} }
@Test @Test
public void customizersForTestClassesWithDifferentSpockMetadataAreEqual() { public void customizersForTestClassesWithDifferentSpockFrameworkAnnotationsAreEqual() {
assertThat(new ImportsContextCustomizer(FirstSpockAnnotatedTestClass.class)) assertThat(
new ImportsContextCustomizer(FirstSpockFrameworkAnnotatedTestClass.class))
.isEqualTo(new ImportsContextCustomizer(
SecondSpockFrameworkAnnotatedTestClass.class));
}
@Test
public void customizersForTestClassesWithDifferentSpockLangAnnotationsAreEqual() {
assertThat(new ImportsContextCustomizer(FirstSpockLangAnnotatedTestClass.class))
.isEqualTo(new ImportsContextCustomizer( .isEqualTo(new ImportsContextCustomizer(
SecondSpockAnnotatedTestClass.class)); SecondSpockLangAnnotatedTestClass.class));
} }
@Metadata(d2 = "foo") @Metadata(d2 = "foo")
@ -54,12 +64,22 @@ public class ImportsContextCustomizerTests {
} }
@SpecMetadata(filename = "foo", line = 10) @SpecMetadata(filename = "foo", line = 10)
static class FirstSpockAnnotatedTestClass { static class FirstSpockFrameworkAnnotatedTestClass {
} }
@SpecMetadata(filename = "bar", line = 10) @SpecMetadata(filename = "bar", line = 10)
static class SecondSpockAnnotatedTestClass { static class SecondSpockFrameworkAnnotatedTestClass {
}
@Stepwise
static class FirstSpockLangAnnotatedTestClass {
}
@Issue("1234")
static class SecondSpockLangAnnotatedTestClass {
} }

Loading…
Cancel
Save