pants: use __defaults__ to propogate unit,integration,benchmarks tags on tests (#5853)

cognifloyd · web-flow · commit feef8d2f0af6 · 2022-12-30T13:00:39.000-06:00
* use __defaults__ to propogate unit,integration,benchmarks tags on tests

* update changelog entry
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -14,7 +14,7 @@ Added
   working on StackStorm, improve our security posture, and improve CI reliability thanks in part
   to pants' use of PEX lockfiles. This is not a user-facing addition.
   #5778 #5789 #5817 #5795 #5830 #5833 #5834 #5841 #5840 #5838 #5842 #5837 #5849 #5850
-  #5846
+  #5846 #5853
   Contributed by @cognifloyd
 
 * Added a joint index to solve the problem of slow mongo queries for scheduled executions. #5805
diff --git a/contrib/runners/action_chain_runner/tests/integration/BUILD b/contrib/runners/action_chain_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/action_chain_runner/tests/unit/BUILD b/contrib/runners/action_chain_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/announcement_runner/tests/integration/BUILD b/contrib/runners/announcement_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/announcement_runner/tests/unit/BUILD b/contrib/runners/announcement_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/http_runner/tests/integration/BUILD b/contrib/runners/http_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/http_runner/tests/unit/BUILD b/contrib/runners/http_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/inquirer_runner/tests/integration/BUILD b/contrib/runners/inquirer_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/inquirer_runner/tests/unit/BUILD b/contrib/runners/inquirer_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/local_runner/tests/integration/BUILD b/contrib/runners/local_runner/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/local_runner/tests/unit/BUILD b/contrib/runners/local_runner/tests/unit/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
diff --git a/contrib/runners/noop_runner/tests/integration/BUILD b/contrib/runners/noop_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/noop_runner/tests/unit/BUILD b/contrib/runners/noop_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/orquesta_runner/tests/integration/BUILD b/contrib/runners/orquesta_runner/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/orquesta_runner/tests/unit/BUILD b/contrib/runners/orquesta_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/contrib/runners/python_runner/tests/integration/BUILD b/contrib/runners/python_runner/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/python_runner/tests/unit/BUILD b/contrib/runners/python_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/contrib/runners/remote_runner/tests/integration/BUILD b/contrib/runners/remote_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/remote_runner/tests/unit/BUILD b/contrib/runners/remote_runner/tests/unit/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
diff --git a/contrib/runners/winrm_runner/tests/integration/BUILD b/contrib/runners/winrm_runner/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/contrib/runners/winrm_runner/tests/unit/BUILD b/contrib/runners/winrm_runner/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/st2actions/tests/integration/BUILD b/st2actions/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/st2actions/tests/unit/BUILD b/st2actions/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/st2api/tests/integration/BUILD b/st2api/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/st2api/tests/unit/BUILD b/st2api/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/st2auth/tests/integration/BUILD b/st2auth/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/st2auth/tests/unit/BUILD b/st2auth/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/st2client/tests/integration/BUILD b/st2client/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/st2client/tests/unit/BUILD b/st2client/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/st2common/benchmarks/BUILD b/st2common/benchmarks/BUILD
@@ -0,0 +1,3 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["benchmarks"])},
+)
diff --git a/st2common/tests/integration/BUILD b/st2common/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_sources()
 
 python_tests(
diff --git a/st2common/tests/unit/BUILD b/st2common/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/st2reactor/tests/integration/BUILD b/st2reactor/tests/integration/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
     dependencies=[
diff --git a/st2reactor/tests/unit/BUILD b/st2reactor/tests/unit/BUILD
@@ -1,3 +1,8 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
+
 python_tests(
     name="tests",
 )
diff --git a/st2stream/tests/integration/BUILD b/st2stream/tests/integration/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
+    extend=True,
+)
diff --git a/st2stream/tests/unit/BUILD b/st2stream/tests/unit/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)
diff --git a/st2tests/integration/BUILD b/st2tests/integration/BUILD
@@ -1,4 +1,5 @@
 __defaults__(
+    {(python_test, python_tests): dict(tags=["integration"])},
     all=dict(
         skip_pylint=True,
     ),
diff --git a/st2tests/tests/unit/BUILD b/st2tests/tests/unit/BUILD
@@ -0,0 +1,4 @@
+__defaults__(
+    {(python_test, python_tests): dict(tags=["unit"])},
+    extend=True,
+)

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,8 @@`
	`1`	`+__defaults__(`
	`2`	`+ {(python_test, python_tests): dict(tags=["unit"])},`
	`3`	`+ extend=True,`
	`4`	`+)`
	`5`	`+`
`1`	`6`	`python_tests(`
`2`	`7`	`name="tests",`
`3`	`8`	`)`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+__defaults__(`
	`2`	`+ {(python_test, python_tests): dict(tags=["benchmarks"])},`
	`3`	`+)`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`__defaults__(`
	`2`	`+ {(python_test, python_tests): dict(tags=["integration"])},`
`2`	`3`	`all=dict(`
`3`	`4`	`skip_pylint=True,`
`4`	`5`	`),`