diff --git a/oap-http/oap-http-test/src/main/java/oap/http/test/HttpAsserts.java b/oap-http/oap-http-test/src/main/java/oap/http/test/HttpAsserts.java index 8e5b41aab..18d6336ab 100644 --- a/oap-http/oap-http-test/src/main/java/oap/http/test/HttpAsserts.java +++ b/oap-http/oap-http-test/src/main/java/oap/http/test/HttpAsserts.java @@ -49,6 +49,7 @@ import static oap.http.test.HttpAsserts.HttpAssertion.assertHttpResponse; import static oap.http.test.HttpAsserts.JsonHttpAssertion.assertJsonResponse; import static oap.io.content.ContentReader.ofString; +import static oap.testng.Asserts.assertString; import static oap.testng.Asserts.contentOfTestResource; import static org.assertj.core.api.Assertions.assertThat; @@ -220,52 +221,52 @@ public HttpAssertion isJson( String json ) { } public HttpAssertion hasReason( String reasonPhrase ) { - assertThat( response.reasonPhrase ).isEqualTo( reasonPhrase ); + assertString( response.reasonPhrase ).isEqualTo( reasonPhrase ); return this; } public HttpAssertion reasonContains( String values ) { - assertThat( response.reasonPhrase ).contains( values ); + assertString( response.reasonPhrase ).contains( values ); return this; } public HttpAssertion reasonContainsPattern( Pattern pattern ) { - assertThat( response.reasonPhrase ).containsPattern( pattern ); + assertString( response.reasonPhrase ).containsPattern( pattern ); return this; } public HttpAssertion reasonContainsPattern( String pattern ) { - assertThat( response.reasonPhrase ).containsPattern( pattern ); + assertString( response.reasonPhrase ).containsPattern( pattern ); return this; } public HttpAssertion hasContentType( String contentType ) { - assertThat( response.contentType ).isEqualTo( contentType ); + assertString( response.contentType ).isEqualTo( contentType ); return this; } public HttpAssertion hasBody( String body ) { - assertThat( response.contentString() ).isEqualTo( body ); + assertString( response.contentString() ).isEqualTo( body ); return this; } public HttpAssertion bodyContains( String values ) { - assertThat( response.contentString() ).contains( values ); + assertString( response.contentString() ).contains( values ); return this; } public HttpAssertion bodyContainsPattern( Pattern pattern ) { - assertThat( response.contentString() ).containsPattern( pattern ); + assertString( response.contentString() ).containsPattern( pattern ); return this; } public HttpAssertion bodyContainsPattern( String pattern ) { - assertThat( response.contentString() ).containsPattern( pattern ); + assertString( response.contentString() ).containsPattern( pattern ); return this; } public HttpAssertion containsHeader( String name, String value ) { - assertThat( response.header( name ).orElse( null ) ).isEqualTo( value ); + assertString( response.header( name ).orElse( null ) ).isEqualTo( value ); return this; } @@ -362,7 +363,7 @@ public static CookieHttpAssertion assertCookie( Cookie cookie ) { } public CookieHttpAssertion hasValue( String value ) { - assertThat( cookie.getValue() ).isEqualTo( value ); + assertString( cookie.getValue() ).isEqualTo( value ); return this; } @@ -371,7 +372,7 @@ public CookieHttpAssertion hasValue( Object value ) { } public CookieHttpAssertion hasDomain( String domain ) { - assertThat( cookie.getDomain() ).isEqualTo( domain ); + assertString( cookie.getDomain() ).isEqualTo( domain ); return this; } @@ -386,7 +387,7 @@ public CookieHttpAssertion expiresAfter( DateTime expiration ) { } public CookieHttpAssertion hasPath( String path ) { - assertThat( cookie.getPath() ).isEqualTo( path ); + assertString( cookie.getPath() ).isEqualTo( path ); return this; } diff --git a/oap-stdlib-test/src/main/java/oap/testng/Asserts.java b/oap-stdlib-test/src/main/java/oap/testng/Asserts.java index ba0cbe728..f75040757 100644 --- a/oap-stdlib-test/src/main/java/oap/testng/Asserts.java +++ b/oap-stdlib-test/src/main/java/oap/testng/Asserts.java @@ -206,7 +206,6 @@ public static String locationOfTestResource( Class contetClass, @Nonnull Stri return "/" + contetClass.getName().replace( ".", "/" ) + ( resource.startsWith( "/" ) ? "" : "/" ) + resource; } - @Deprecated public static class StringAssertion extends AbstractCharSequenceAssert { protected StringAssertion( CharSequence value ) { super( value, StringAssertion.class ); diff --git a/oap-storage/oap-storage-cloud/src/main/java/oap/storage/cloud/FileSystem.java b/oap-storage/oap-storage-cloud/src/main/java/oap/storage/cloud/FileSystem.java index 722e5e8d0..4ff754888 100644 --- a/oap-storage/oap-storage-cloud/src/main/java/oap/storage/cloud/FileSystem.java +++ b/oap-storage/oap-storage-cloud/src/main/java/oap/storage/cloud/FileSystem.java @@ -35,11 +35,6 @@ public class FileSystem implements AutoCloseable { private static final HashMap> providers = new HashMap<>(); - private static final Cache apis = CacheBuilder - .newBuilder() - .removalListener( rl -> Closeables.close( ( FileSystemCloudApi ) rl.getValue() ) ) - .build(); - static { try { List urls = Resources.urls( FileSystem.class, "/cloud-service.properties" ); @@ -63,6 +58,10 @@ public class FileSystem implements AutoCloseable { } public final FileSystemConfiguration fileSystemConfiguration; + private final Cache apis = CacheBuilder + .newBuilder() + .removalListener( rl -> Closeables.close( ( FileSystemCloudApi ) rl.getValue() ) ) + .build(); public FileSystem( FileSystemConfiguration fileSystemConfiguration ) { this.fileSystemConfiguration = fileSystemConfiguration; diff --git a/pom.xml b/pom.xml index 6f164b2e5..5748e7d48 100644 --- a/pom.xml +++ b/pom.xml @@ -71,7 +71,7 @@ - 25.0.13 + 25.0.14 22.0.1 25.0.0