Remove upcasting test in BridgingTest.cpp #37519
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
While these tests currently happen to pass, actually trying to
instantiate
fromJs
fails to compile. I suspect this is attributableto some difference in how
fromJs
is instantiated in evaluated andunevaluated contexts. That is since
supportsFromJs
only instantiatesit in an unevaluated context (in a decltype), the rules are presumably
different.
It is also worth noting that the operator of up-casting JSI types to
jsi::Value
is explicitly deleted inConverter
, which suggeststhat the conversion this test is checking for may be intentionally
unsupported.
For now, since
fromJs
cannot actually be used with the givenparameters, delete the test.
This unblocks a later diff which changes the constructor of
jsi::Value
such thatstd::is_convertible<jsi::Object &, jsi::Value>
is no longer true (theconversion is never allowed, but is currently enforced by
static_assert
). With that changesupportsFromJs<jsi::Value, jsi::Object>
also becomes false.Reviewed By: javache, cortinico
Differential Revision: D46059603