Skip to content

Commit

Permalink
cosmetic: eliminate some code warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
stefanseifert committed Jul 4, 2024
1 parent a7fe15d commit a170845
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,6 @@ public boolean useAdobeStandardNames() {
// switch to adobe standard naming
return true;
}
};
}

}
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ public List<Class<? extends MediaProcessor>> getPostProcessors() {
return List.of(TestPostProcessor.class);
}

};
}


@Model(adaptables = {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
import org.mockito.Mock.Strictness;
import org.mockito.junit.jupiter.MockitoExtension;

import io.wcm.handler.media.MediaArgs;
Expand All @@ -42,7 +43,7 @@
@ExtendWith(MockitoExtension.class)
class CroppingRatiosTest {

@Mock(lenient = true)
@Mock(strictness = Strictness.LENIENT)
private MediaFormatHandler mediaFormatHandler;
private SortedSet<MediaFormat> mediaFormats = new TreeSet<>();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,6 @@ public enum DummyAppTemplate {
this.resourceType = resourceTypeFromPath;
}

DummyAppTemplate(String templatePath, String resourceType) {
this.templatePath = templatePath;
this.resourceType = resourceType;
}

/**
* Template path
* @return Path
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ public class DummyMediaHandlerConfig extends MediaHandlerConfig {
}

@Override
@Deprecated
public boolean includeAssetWebRenditionsByDefault() {
// unit tests rely on old behavior
return true;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@ void testWithAllowSpecificRenditions() {
void testWithDisallowWebRenditionsViaMediaHandlerConfig() {
context.registerService(MediaHandlerConfig.class, new MediaHandlerConfig() {
@Override
@Deprecated
public boolean includeAssetWebRenditionsByDefault() {
return false;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,14 @@

import static io.wcm.handler.mediasource.ngdm.impl.NextGenDynamicMediaReferenceSample.SAMPLE_ASSET_ID;

import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;

/**
* Next Gen Dynamic Media Metadata samples.
*/
public final class MetadataSample {

@SuppressFBWarnings("HSC_HUGE_SHARED_STRING_CONSTANT")
public static final String METADATA_JSON_IMAGE = "{"
+ " \"assetId\": \"" + SAMPLE_ASSET_ID + "\","
+ " \"repositoryMetadata\": {"
Expand Down

0 comments on commit a170845

Please sign in to comment.