Skip to content

Commit

Permalink
Rename oppia_android_module_level_test.
Browse files Browse the repository at this point in the history
These are layer-level tests, not 'module' tests as that's a Gradle
concept that doesn't make sense in a Bazel-only world (and only adds
confusion).
  • Loading branch information
BenHenning committed Feb 26, 2025
1 parent 2056b44 commit a7e4734
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 11 deletions.
4 changes: 2 additions & 2 deletions app/app_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Macros for app layer tests.
"""

load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
load("//:oppia_android_test.bzl", "oppia_android_layer_level_test")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def app_test(
Expand All @@ -28,7 +28,7 @@ def app_test(
filtered_tests: list of str. The test files that should not have tests defined for them.
**kwargs: additional parameters passed in.
"""
oppia_android_module_level_test(
oppia_android_layer_level_test(
name = name,
filtered_tests = filtered_tests,
deps = deps,
Expand Down
4 changes: 2 additions & 2 deletions data/data_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Macros for data tests.
"""

load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
load("//:oppia_android_test.bzl", "oppia_android_layer_level_test")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def data_test(name, filtered_tests, deps, **kwargs):
Expand All @@ -15,7 +15,7 @@ def data_test(name, filtered_tests, deps, **kwargs):
deps: list of str. The list of dependencies needed to build and run this test.
**kwargs: additional parameters passed in.
"""
oppia_android_module_level_test(
oppia_android_layer_level_test(
name = name,
filtered_tests = filtered_tests,
deps = deps,
Expand Down
4 changes: 2 additions & 2 deletions domain/domain_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Macros for domain layer tests.
"""

load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
load("//:oppia_android_test.bzl", "oppia_android_layer_level_test")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def domain_test(name, filtered_tests, deps):
Expand All @@ -15,7 +15,7 @@ def domain_test(name, filtered_tests, deps):
deps: list of str. The list of dependencies needed to build and run this test.
"""

oppia_android_module_level_test(
oppia_android_layer_level_test(
name = name,
filtered_tests = filtered_tests,
deps = deps,
Expand Down
2 changes: 1 addition & 1 deletion oppia_android_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Central macros pertaining to setting up tests across the codebase.
load("@io_bazel_rules_kotlin//kotlin:android.bzl", "kt_android_library")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def oppia_android_module_level_test(
def oppia_android_layer_level_test(
name,
filtered_tests,
deps,
Expand Down
4 changes: 2 additions & 2 deletions testing/testing_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Macros for testing tests.
"""

load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
load("//:oppia_android_test.bzl", "oppia_android_layer_level_test")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def testing_test(name, filtered_tests, deps):
Expand All @@ -14,7 +14,7 @@ def testing_test(name, filtered_tests, deps):
filtered_tests: list of str. The test files that should not have tests defined for them.
deps: list of str. The list of dependencies needed to build and run this test.
"""
oppia_android_module_level_test(
oppia_android_layer_level_test(
name = name,
filtered_tests = filtered_tests,
deps = deps,
Expand Down
4 changes: 2 additions & 2 deletions utility/utility_test.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Macros for utility layer tests.
"""

load("//:oppia_android_test.bzl", "oppia_android_module_level_test")
load("//:oppia_android_test.bzl", "oppia_android_layer_level_test")

# TODO(#1620): Remove layer-specific test macros once Gradle is removed
def utility_test(name, filtered_tests, deps):
Expand All @@ -14,7 +14,7 @@ def utility_test(name, filtered_tests, deps):
filtered_tests: list of str. The test files that should not have tests defined for them.
deps: list of str. The list of dependencies needed to build and run this test.
"""
oppia_android_module_level_test(
oppia_android_layer_level_test(
name = name,
filtered_tests = filtered_tests,
deps = deps,
Expand Down

0 comments on commit a7e4734

Please sign in to comment.