Add test to check spec directories
When submitting specs one very common mistake is to submit it to specs/liberty/ instead of specs/liberty/approved/. Add a new test to make sure submitted specs are not in the wrong directory. Test output: MismatchError: 'foo.rst' not in ['redirects', 'implemented', 'approved']: Found unexpected file in 'specs/liberty', specs should be submitted to 'specs/liberty/approved' Change-Id: I27a532dc0b85d712698821d77555d2aeb57e83a1
This commit is contained in:
parent
0987cd41eb
commit
445ea94655
|
@ -0,0 +1,32 @@
|
|||
# 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.
|
||||
|
||||
import glob
|
||||
import os
|
||||
|
||||
import testtools
|
||||
|
||||
|
||||
class TestDirectories(testtools.TestCase):
|
||||
|
||||
def test_directories(self):
|
||||
releases = [x.split('/')[1] for x in glob.glob('specs/*/')]
|
||||
for release in releases:
|
||||
files = os.listdir("specs/%s/" % release)
|
||||
valid = ['redirects', 'implemented', 'approved']
|
||||
for name in files:
|
||||
if name.startswith('.'):
|
||||
continue
|
||||
self.assertIn(name, valid,
|
||||
"Found unexpected file in "
|
||||
"'specs/%s', specs should be submitted to "
|
||||
"'specs/%s/approved'" % (release, release))
|
Loading…
Reference in New Issue