Marked as WIP - still need to do a cleanup pass and run a few more tests with both appium and detox.

@NathanWalker seems quite good to me.
One question i have is are we sure this:

if (typeof __USE_TEST_ID__ !== 'undefined' && __USE_TEST_ID__ && this.testID) {
...
}

is removed if __USE_TEST_ID__ is not defined and is transformed to if (this.testID) {..} if set ?

It is worth testing to ensure the code is removed completely

@farfromrefug I'll be checking that - just got a detox demo set up, still need an appium one to check all cases - and then clean up anything that can be cleaned up...

If that doesn't get tree-shaken, I'll just drop the typeof check - since we'll be setting it in webpack by default to false (unless --env.e2e).

@farfromrefug seems to be tree-shaken away just fine!

// default

    [_view_common__WEBPACK_IMPORTED_MODULE_0__.testIDProperty.setNative](value) {
        this.setTestID(this.nativeViewProtected, value);
    }
    setTestID(view, value) {
        if (false) {}
    }

// with --env.e2e

    [_view_common__WEBPACK_IMPORTED_MODULE_0__.testIDProperty.setNative](value) {
        this.setTestID(this.nativeViewProtected, value);
    }
    setTestID(view, value) {
        if (true) {
            const id = _utils__WEBPACK_IMPORTED_MODULE_11__.ad.resources.getId(':id/nativescript_accessibility_id');
            if (id) {
                view.setTag(id, value);
                view.setTag(value);
            }
            view.setContentDescription(value);
        }
    }

@rigor789 very good! very good to know we write something like this 😃