<?xml version="1.0"?>
<!--
  Copyright 2021 Flyte Authors.

  Licensed under the Apache License, Version 2.0 (the "License");
  you may not use this file except in compliance with the License.
  You may obtain a copy of the License at

      http://www.apache.org/licenses/LICENSE-2.0

  Unless required by applicable law or agreed to in writing,
  software distributed under the License is distributed on an
  "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
  KIND, either express or implied.  See the License for the
  specific language governing permissions and limitations
  under the License.
-->
<FindBugsFilter>

  <Match>
    <Class name="~.*\.*AutoValue_.*"/>
  </Match>

  <!-- Filter flyteidl protos generated classes -->
  <Match>
    <Package name="~flyteidl\b.*"/>
  </Match>

  <!-- Filter protoc_gen_swagger protos generated classes -->
  <Match>
    <Package name="~grpc.gateway.protoc_gen_swagger.options\b.*"/>
  </Match>

  <!-- Don't need serialVersionUID in examples -->
  <Match>
    <Package name="~.*examples.*"/>
    <Bug pattern="SE_NO_SERIALVERSIONID"/>
  </Match>

  <!-- Often test assert exceptions by calling a method and ignoring the result  -->
  <Match>
    <Class name="~.*Test$"/>
    <Bug pattern="RV_RETURN_VALUE_IGNORED_INFERRED"/>
  </Match>

  <!-- Test initialize fields in Setup, no in constructor  -->
  <Match>
    <Class name="~.*Test$"/>
    <Bug pattern="UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR"/>
  </Match>

  <!-- Non-static anon classes are fine in tests -->
  <Match>
    <Class name="~.*Test$"/>
    <Bug pattern="SIC_INNER_SHOULD_BE_STATIC_ANON"/>
  </Match>

  <!-- See: https://github.com/spotbugs/spotbugs/issues/600 -->
  <Match>
    <Bug pattern="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE"/>
  </Match>

  <Match>
    <Bug pattern="RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE"/>
  </Match>

</FindBugsFilter>