This is an archive of the discontinued LLVM Phabricator instance.

Tests no longer need the 'REQUIRES: SHELL' line.
ClosedPublic

Authored by angelgarcia on Aug 24 2015, 6:45 AM.

Reviewers

Commits

Summary

Update python script, so that it doesn't print that line in new tests.

angelgarcia retitled this revision from to Tests no longer need the 'REQUIRES: SHELL' line..

angelgarcia updated this object.

angelgarcia added a reviewer: alexfh.

LG. Thank you!

This revision is now accepted and ready to land.Aug 24 2015, 3:33 PM

Path

Size

clang-tidy/

1 line

	Show First 20 Lines • Show All 178 Lines • ▼ Show 20 Lines
	# Adds a test for the check.			# Adds a test for the check.
	def write_test(module_path, module, check_name):			def write_test(module_path, module, check_name):
	check_name_dashes = module + '-' + check_name			check_name_dashes = module + '-' + check_name
	filename = os.path.join(module_path, '../../test/clang-tidy',			filename = os.path.join(module_path, '../../test/clang-tidy',
	check_name_dashes + '.cpp')			check_name_dashes + '.cpp')
	with open(filename, 'w') as f:			with open(filename, 'w') as f:
	f.write(			f.write(
	"""// RUN: %%python %%S/check_clang_tidy.py %%s %(check_name_dashes)s %%t			"""// RUN: %%python %%S/check_clang_tidy.py %%s %(check_name_dashes)s %%t
	// REQUIRES: shell

	// FIXME: Add something that triggers the check here.			// FIXME: Add something that triggers the check here.
	void f();			void f();
	// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: function 'f' is insufficiently awesome [%(check_name_dashes)s]			// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: function 'f' is insufficiently awesome [%(check_name_dashes)s]

	// FIXME: Verify the applied fix.			// FIXME: Verify the applied fix.
	// * Make the CHECK patterns specific enough and try to make verified lines			// * Make the CHECK patterns specific enough and try to make verified lines
	// unique to avoid incorrect matches.			// unique to avoid incorrect matches.
	Show All 30 Lines