Fix incomplete snapshots

astral-sh · Mar 16, 2023 · 006a96f · 006a96f
1 parent aa51ece
commit 006a96f
Show file tree

Hide file tree

Showing 27 changed files with 1,435 additions and 50 deletions.
diff --git a/...c/rules/flake8_pytest_style/snapshots/ruff__rules__flake8_pytest_style__tests__PT018.snap b/...c/rules/flake8_pytest_style/snapshots/ruff__rules__flake8_pytest_style__tests__PT018.snap
@@ -62,6 +62,26 @@ expression: diagnostics
       row: 17
       column: 0
   parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: Break down assertion into multiple parts
+    fixable: true
+  location:
+    row: 16
+    column: 4
+  end_location:
+    row: 16
+    column: 39
+  fix:
+    content: "    assert something\n    assert something_else\n"
+    location:
+      row: 16
+      column: 0
+    end_location:
+      row: 17
+      column: 0
+  parent: ~
 - kind:
     name: CompositeAssertion
     body: Assertion should be broken down into multiple parts
@@ -254,4 +274,134 @@ expression: diagnostics
     column: 5
   fix: ~
   parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: Break down assertion into multiple parts
+    fixable: true
+  location:
+    row: 37
+    column: 4
+  end_location:
+    row: 37
+    column: 20
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: Break down assertion into multiple parts
+    fixable: true
+  location:
+    row: 38
+    column: 4
+  end_location:
+    row: 38
+    column: 20
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 40
+    column: 4
+  end_location:
+    row: 40
+    column: 56
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 41
+    column: 4
+  end_location:
+    row: 41
+    column: 80
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 41
+    column: 4
+  end_location:
+    row: 41
+    column: 56
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 42
+    column: 4
+  end_location:
+    row: 42
+    column: 80
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 43
+    column: 4
+  end_location:
+    row: 43
+    column: 64
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: ~
+    fixable: false
+  location:
+    row: 44
+    column: 4
+  end_location:
+    row: 44
+    column: 64
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: Break down assertion into multiple parts
+    fixable: true
+  location:
+    row: 45
+    column: 4
+  end_location:
+    row: 51
+    column: 5
+  fix: ~
+  parent: ~
+- kind:
+    name: CompositeAssertion
+    body: Assertion should be broken down into multiple parts
+    suggestion: Break down assertion into multiple parts
+    fixable: true
+  location:
+    row: 46
+    column: 4
+  end_location:
+    row: 52
+    column: 5
+  fix: ~
+  parent: ~
 
diff --git a/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM102_SIM102.py.snap b/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM102_SIM102.py.snap
@@ -22,6 +22,26 @@ expression: diagnostics
       row: 5
       column: 0
   parent: ~
+- kind:
+    name: CollapsibleIf
+    body: "Use a single `if` statement instead of nested `if` statements"
+    suggestion: "Combine `if` statements using `and`"
+    fixable: true
+  location:
+    row: 6
+    column: 0
+  end_location:
+    row: 7
+    column: 9
+  fix:
+    content: "if a and b and c:\n    d\n"
+    location:
+      row: 6
+      column: 0
+    end_location:
+      row: 9
+      column: 0
+  parent: ~
 - kind:
     name: CollapsibleIf
     body: "Use a single `if` statement instead of nested `if` statements"
@@ -62,6 +82,32 @@ expression: diagnostics
       row: 18
       column: 0
   parent: ~
+- kind:
+    name: CollapsibleIf
+    body: "Use a single `if` statement instead of nested `if` statements"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 16
+    column: 0
+  end_location:
+    row: 18
+    column: 9
+  fix: ~
+  parent: ~
+- kind:
+    name: CollapsibleIf
+    body: "Use a single `if` statement instead of nested `if` statements"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 17
+    column: 0
+  end_location:
+    row: 19
+    column: 9
+  fix: ~
+  parent: ~
 - kind:
     name: CollapsibleIf
     body: "Use a single `if` statement instead of nested `if` statements"

diff --git a/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM103_SIM103.py.snap b/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM103_SIM103.py.snap
@@ -82,6 +82,19 @@ expression: diagnostics
       row: 35
       column: 24
   parent: ~
+- kind:
+    name: NeedlessBool
+    body: "Return the condition `a` directly"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 45
+    column: 4
+  end_location:
+    row: 48
+    column: 19
+  fix: ~
+  parent: ~
 - kind:
     name: NeedlessBool
     body: "Return the condition `a` directly"
@@ -95,6 +108,19 @@ expression: diagnostics
     column: 19
   fix: ~
   parent: ~
+- kind:
+    name: NeedlessBool
+    body: "Return the condition `a` directly"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 71
+    column: 4
+  end_location:
+    row: 74
+    column: 20
+  fix: ~
+  parent: ~
 - kind:
     name: NeedlessBool
     body: "Return the condition `a` directly"

diff --git a/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM108_SIM108.py.snap b/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM108_SIM108.py.snap
@@ -22,6 +22,19 @@ expression: diagnostics
       row: 5
       column: 9
   parent: ~
+- kind:
+    name: UseTernaryOperator
+    body: "Use ternary operator `abc = x if x > 0 else -x` instead of `if`-`else`-block"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 55
+    column: 0
+  end_location:
+    row: 60
+    column: 12
+  fix: ~
+  parent: ~
 - kind:
     name: UseTernaryOperator
     body: "Use ternary operator `abc = x if x > 0 else -x` instead of `if`-`else`-block"
@@ -55,6 +68,19 @@ expression: diagnostics
       row: 85
       column: 45
   parent: ~
+- kind:
+    name: UseTernaryOperator
+    body: "Use ternary operator `exitcode = 0 if True else 1` instead of `if`-`else`-block"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 91
+    column: 0
+  end_location:
+    row: 94
+    column: 16
+  fix: ~
+  parent: ~
 - kind:
     name: UseTernaryOperator
     body: "Use ternary operator `exitcode = 0 if True else 1` instead of `if`-`else`-block"
@@ -68,6 +94,32 @@ expression: diagnostics
     column: 16
   fix: ~
   parent: ~
+- kind:
+    name: UseTernaryOperator
+    body: "Use ternary operator `x = 3 if True else 5` instead of `if`-`else`-block"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 98
+    column: 0
+  end_location:
+    row: 99
+    column: 11
+  fix: ~
+  parent: ~
+- kind:
+    name: UseTernaryOperator
+    body: "Use ternary operator `x = 3 if True else 5` instead of `if`-`else`-block"
+    suggestion: ~
+    fixable: false
+  location:
+    row: 103
+    column: 0
+  end_location:
+    row: 106
+    column: 9
+  fix: ~
+  parent: ~
 - kind:
     name: UseTernaryOperator
     body: "Use ternary operator `x = 3 if True else 5` instead of `if`-`else`-block"

diff --git a/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM110_SIM110.py.snap b/...ules/flake8_simplify/snapshots/ruff__rules__flake8_simplify__tests__SIM110_SIM110.py.snap
@@ -142,6 +142,32 @@ expression: diagnostics
       row: 87
       column: 19
   parent: ~
+- kind:
+    name: ReimplementedBuiltin
+    body: "Use `return any(check(x) for x in iterable)` instead of `for` loop"
+    suggestion: "Replace with `return any(check(x) for x in iterable)`"
+    fixable: true
+  location:
+    row: 99
+    column: 4
+  end_location:
+    row: 101
+    column: 23
+  fix: ~
+  parent: ~
+- kind:
+    name: ReimplementedBuiltin
+    body: "Use `return all(not check(x) for x in iterable)` instead of `for` loop"
+    suggestion: "Replace with `return all(not check(x) for x in iterable)`"
+    fixable: true
+  location:
+    row: 109
+    column: 4
+  end_location:
+    row: 111
+    column: 24
+  fix: ~
+  parent: ~
 - kind:
     name: ReimplementedBuiltin
     body: "Use `return any(check(x) for x in iterable)` instead of `for` loop"