Conversation
| doc = "A dictionary of environment variables set for the execution of the test. Will be subject to make variable and $(location) expansion.", | ||
| ), | ||
| robolectric_properties_file = attr.string( | ||
| robolectric_properties_file = attr.label( |
There was a problem hiding this comment.
You can also just look up the properties file in the runfiles and error out if they aren't found. I believe this is how the native Android rules handle this use case already.
There was a problem hiding this comment.
Thanks for the example. But if user have the option to use their own properties file, would this look-up approach work? On the other hand, I'm not sure if users ever want to use their own properties files.
There was a problem hiding this comment.
It should as long as you are writing the file to the expected location. Probably isn't really a major use case that people will have though.
One use case that comes to mind would be stripping out old Android versions that you no longer want to support from Robolectric. If there is a need to customize the properties file we can always build out support for that in rules_robolectric.
5abd169 to
ab64bec
Compare
TEST bazelci @Head