De-bazel filename_test. Part of the effort to remove all_opensource_files.
PiperOrigin-RevId: 184021942
This commit is contained in:
parent
b79c3b2d1e
commit
099c91b506
@ -517,9 +517,14 @@ do_check_futures_test() {
|
|||||||
python check_futures_test.py
|
python check_futures_test.py
|
||||||
}
|
}
|
||||||
|
|
||||||
|
do_check_file_name_test() {
|
||||||
|
cd "$ROOT_DIR/tensorflow/tools/test"
|
||||||
|
python file_name_test.py
|
||||||
|
}
|
||||||
|
|
||||||
# Supply all sanity step commands and descriptions
|
# Supply all sanity step commands and descriptions
|
||||||
SANITY_STEPS=("do_pylint PYTHON2" "do_pylint PYTHON3" "do_check_futures_test" "do_buildifier" "do_bazel_nobuild" "do_pip_package_licenses_check" "do_lib_package_licenses_check" "do_java_package_licenses_check" "do_pip_smoke_test" "do_check_load_py_test" "do_code_link_check" "do_cmake_python_sanity")
|
SANITY_STEPS=("do_pylint PYTHON2" "do_pylint PYTHON3" "do_check_futures_test" "do_buildifier" "do_bazel_nobuild" "do_pip_package_licenses_check" "do_lib_package_licenses_check" "do_java_package_licenses_check" "do_pip_smoke_test" "do_check_load_py_test" "do_code_link_check" "do_cmake_python_sanity" "do_check_file_name_test")
|
||||||
SANITY_STEPS_DESC=("Python 2 pylint" "Python 3 pylint" "Check that python files have certain __future__ imports" "buildifier check" "bazel nobuild" "pip: license check for external dependencies" "C library: license check for external dependencies" "Java Native Library: license check for external dependencies" "Pip Smoke Test: Checking py_test dependencies exist in pip package" "Check load py_test: Check that BUILD files with py_test target properly load py_test" "Code Link Check: Check there are no broken links" "Test entries in /tensorflow/contrib/cmake/python_{modules|protos|protos_cc}.txt for validity and consistency")
|
SANITY_STEPS_DESC=("Python 2 pylint" "Python 3 pylint" "Check that python files have certain __future__ imports" "buildifier check" "bazel nobuild" "pip: license check for external dependencies" "C library: license check for external dependencies" "Java Native Library: license check for external dependencies" "Pip Smoke Test: Checking py_test dependencies exist in pip package" "Check load py_test: Check that BUILD files with py_test target properly load py_test" "Code Link Check: Check there are no broken links" "Test entries in /tensorflow/contrib/cmake/python_{modules|protos|protos_cc}.txt for validity and consistency" "Check file names for cases")
|
||||||
|
|
||||||
INCREMENTAL_FLAG=""
|
INCREMENTAL_FLAG=""
|
||||||
DEFAULT_BAZEL_CONFIGS="--config=hdfs --config=gcp"
|
DEFAULT_BAZEL_CONFIGS="--config=hdfs --config=gcp"
|
||||||
|
48
tensorflow/tools/test/file_name_test.py
Normal file
48
tensorflow/tools/test/file_name_test.py
Normal file
@ -0,0 +1,48 @@
|
|||||||
|
#!/usr/bin/python
|
||||||
|
# Copyright 2017 The TensorFlow Authors. All Rights Reserved.
|
||||||
|
#
|
||||||
|
# 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.
|
||||||
|
# ==============================================================================
|
||||||
|
#
|
||||||
|
# Test that checks if we have any issues with case insensitive filesystems.
|
||||||
|
|
||||||
|
from __future__ import absolute_import
|
||||||
|
from __future__ import division
|
||||||
|
from __future__ import print_function
|
||||||
|
|
||||||
|
import os
|
||||||
|
|
||||||
|
BASE_DIR = os.path.abspath(os.path.join(os.path.dirname(__file__), '../..'))
|
||||||
|
ERROR_MESSAGE = """
|
||||||
|
Files with same name but different case detected in directory: {}
|
||||||
|
"""
|
||||||
|
|
||||||
|
|
||||||
|
def main():
|
||||||
|
# Make sure BASE_DIR ends with tensorflow. If it doesn't, we probably
|
||||||
|
# computed the wrong directory.
|
||||||
|
if os.path.split(BASE_DIR)[-1] != 'tensorflow':
|
||||||
|
raise AssertionError(
|
||||||
|
"BASE_DIR = '%s' doesn't end with tensorflow" % BASE_DIR)
|
||||||
|
|
||||||
|
for dirpath, dirnames, filenames in os.walk(BASE_DIR, followlinks=True):
|
||||||
|
lowercase_directories = [x.lower() for x in dirnames]
|
||||||
|
lowercase_files = [x.lower() for x in filenames]
|
||||||
|
|
||||||
|
lowercase_dir_contents = lowercase_directories + lowercase_files
|
||||||
|
if len(lowercase_dir_contents) != len(set(lowercase_dir_contents)):
|
||||||
|
raise AssertionError(ERROR_MESSAGE.format(dirpath))
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == '__main__':
|
||||||
|
main()
|
Loading…
Reference in New Issue
Block a user