Skip to content

Commit

Permalink
Make the ext-types proc-macro tests pass again
Browse files Browse the repository at this point in the history
This started failing because of #1809.  I don't think that's a new
issue, but somehow the changes in this branch caused the tests to start
failing.

Removing this extra function, which I didn't mean to commit in the first
place, makes the tests pass.  Let's fix it for real in another branch.
  • Loading branch information
bendk committed Oct 27, 2023
1 parent d5eb09c commit 98350a0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
5 changes: 0 additions & 5 deletions fixtures/ext-types/uniffi-one/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,4 @@ fn get_my_proc_macro_type(t: UniffiOneProcMacroType) -> UniffiOneProcMacroType {
t
}

#[uniffi::export]
async fn get_uniffi_one_async() -> UniffiOneEnum {
UniffiOneEnum::One
}

uniffi::include_scaffolding!("uniffi-one");
2 changes: 2 additions & 0 deletions uniffi_bindgen/src/bindings/python/templates/EnumTemplate.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ def read(buf):
{%- endfor %}
raise InternalError("Raw enum value doesn't match any cases")

@staticmethod
def check(value):
{%- if e.variants().is_empty() %}
pass
Expand All @@ -98,6 +99,7 @@ def check(value):
{%- endfor %}
{%- endif %}

@staticmethod
def write(value, buf):
{%- for variant in e.variants() %}
{%- if e.is_flat() %}
Expand Down

0 comments on commit 98350a0

Please sign in to comment.