Normalize case handling for objects and callbacks

Objects and callbacks were not handling case consistently in generated
across code, leading to compilation errors in some cases (see #22).

The main problem arises in capital acronym handling, i.e. `HTTPClient`.
Such acronyms are converted using `heck::to_upper_camel_case`, which
converts upper case acronym `HTTP` into upper camel case `Http`, i.e.
`HttpClient`.

bindgen/templates/CallbackInterfaceTemplate.go

@@ -15,7 +15,7 @@ type {{ type_name }} interface {
 // {{ foreign_callback }} cannot be callable be a compiled function at a same time
 type {{ foreign_callback }} struct {}
 
-{% let cgo_callback_fn = self.cgo_callback_fn(type_name) -%}
+{% let cgo_callback_fn = self.cgo_callback_fn(cbi.name()) -%}
 //export {{ cgo_callback_fn }}
 func {{ cgo_callback_fn }}(handle C.uint64_t, method C.int32_t, args C.RustBuffer, outBuf *C.RustBuffer) C.int32_t {
 	cb := {{ type_|lift_fn }}(uint64(handle));

bindgen/templates/ObjectTemplate.go

@@ -5,23 +5,23 @@
 {{- self.add_import("runtime") }}
 
 {%- let obj = ci.get_object_definition(name).unwrap() %}
-{%- let canonical_name = type_|canonical_name %}
+{%- let obj_name = obj.name()|class_name %}
 {%- if self.include_once_check("ObjectRuntime.go") %}{% include "ObjectRuntime.go" %}{% endif %}
 
-type {{ canonical_name }} struct {
+type {{ obj_name }} struct {
 	ffiObject FfiObject
 }
 
 {%- match obj.primary_constructor() %}
 {%- when Some with (cons) %}
-func New{{ canonical_name }}({% call go::arg_list_decl(cons) -%}) {% call go::return_type_decl(cons) %} {
+func New{{ obj_name }}({% call go::arg_list_decl(cons) -%}) {% call go::return_type_decl(cons) %} {
 	{% call go::ffi_call_binding(func, "") %}
 }
 {%- when None %}
 {%- endmatch %}
 
 {% for cons in obj.alternate_constructors() -%}
-func {{ canonical_name }}{{ cons.name()|fn_name }}({% call go::arg_list_decl(cons) %}) {% call go::return_type_decl(cons) %} {
+func {{ obj_name }}{{ cons.name()|fn_name }}({% call go::arg_list_decl(cons) %}) {% call go::return_type_decl(cons) %} {
 	{% call go::ffi_call_binding(func, "") %}
 }
 {% endfor %}
@@ -44,7 +44,7 @@ type {{ obj|ffi_converter_name }} struct {}
 var {{ obj|ffi_converter_name }}INSTANCE = {{ obj|ffi_converter_name }}{}
 
 func (c {{ obj|ffi_converter_name }}) lift(pointer unsafe.Pointer) {{ type_name }} {
-	result := &{{ canonical_name }} {
+	result := &{{ obj_name }} {
 		newFfiObject(
 			pointer,
 			func(pointer unsafe.Pointer, status *C.RustCallStatus) {

binding_tests/custom_types_test.go

@@ -5,11 +5,11 @@
 package binding_tests
 
 import (
-    "testing"
-    "net/url"
+	"net/url"
+	"testing"
 
-    "github.com/NordSecurity/uniffi-bindgen-go/binding_tests/generated/uniffi/custom_types"
-    "github.com/stretchr/testify/assert"
+	"github.com/NordSecurity/uniffi-bindgen-go/binding_tests/generated/uniffi/custom_types"
+	"github.com/stretchr/testify/assert"
 )
 
 func unwrap[T any](value T, err error) T {
@@ -30,6 +30,6 @@ func TestCustomTypes(t *testing.T) {
 
 	// Change some data and ensure that the round-trip works
 	demo.Url = *unwrap(url.Parse("http://new.example.com/"))
-	demo.Handle = 456;
+	demo.Handle = 456
 	assert.Equal(t, demo, custom_types.GetCustomTypesDemo(&demo))
 }

binding_tests/errors_test.go

@@ -5,9 +5,9 @@
 package binding_tests
 
 import (
-	"github.com/NordSecurity/uniffi-bindgen-go/binding_tests/generated/uniffi/errors"
 	goerrors "errors"
 	"fmt"
+	"github.com/NordSecurity/uniffi-bindgen-go/binding_tests/generated/uniffi/errors"
 	"github.com/stretchr/testify/assert"
 	"testing"
 )

binding_tests/name_case_test.go

@@ -0,0 +1,30 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+package binding_tests
+
+import (
+	"errors"
+	. "github.com/NordSecurity/uniffi-bindgen-go/binding_tests/generated/uniffi/name_case"
+	"testing"
+)
+
+type MyNameCaseCallback struct{}
+
+func (*MyNameCaseCallback) Test() {}
+
+func TestNameCaseCompiles(t *testing.T) {
+	_ = EnumTestVariantOne
+	_ = AssociatedEnumTestVariantTest{0}
+
+	var expectedError *ErrorTestVariantOne
+	errors.As(NewErrorTestVariantOne(), &expectedError)
+
+	var expectedAssociatedError *AssociatedErrorTestVariantTest
+	errors.As(NewAssociatedErrorTestVariantTest(0), &expectedAssociatedError)
+
+	var _ *ObjectTest = NewObjectTest()
+	_ = RecordTest{0}
+	var _ CallbackTest = &MyNameCaseCallback{}
+}

fixtures/Cargo.toml

@@ -20,7 +20,7 @@ uniffi-fixture-callbacks = { path = "../3rd-party/uniffi-rs/fixtures/callbacks"
 uniffi-fixture-coverall = { path = "../3rd-party/uniffi-rs/fixtures/coverall" }
 uniffi-fixture-external-types = { path = "../3rd-party/uniffi-rs/fixtures/external-types/lib" }
 uniffi-fixture-time = { path = "../3rd-party/uniffi-rs/fixtures/uniffi-fixture-time" }
-uniffi-go-fixture-errors = { path = "errors" }
 uniffi-go-fixture-destroy = { path = "destroy" }
+uniffi-go-fixture-errors = { path = "errors" }
+uniffi-go-fixture-name-case = { path = "name-case" }
 uniffi-go-fixture-objects = { path = "objects" }
-

fixtures/name-case/Cargo.toml

@@ -0,0 +1,24 @@
+[package]
+name = "uniffi-go-fixture-name-case"
+version = "0.22.0"
+authors = ["Firefox Sync Team <[email protected]>"]
+edition = "2021"
+license = "MPL-2.0"
+publish = false
+
+[lib]
+name = "uniffi_go_name_case"
+crate-type = ["lib", "cdylib"]
+
+[dependencies]
+thiserror = "1.0"
+uniffi = { path = "../../3rd-party/uniffi-rs/uniffi" }
+
+[build-dependencies]
+uniffi = {path = "../../3rd-party/uniffi-rs/uniffi", features = ["build"] }
+
+[dev-dependencies]
+glob = "0.3"
+uniffi = {path = "../../3rd-party/uniffi-rs/uniffi", features = ["bindgen-tests"] }
+uniffi_bindgen = { path = "../../3rd-party/uniffi-rs/uniffi_bindgen" }
+uniffi_testing = { path = "../../3rd-party/uniffi-rs/uniffi_testing" }

fixtures/name-case/README.md

@@ -0,0 +1,3 @@
+# A basic test for uniffi components
+
+This test covers case senstitivity for generate code.

fixtures/name-case/build.rs

@@ -0,0 +1,7 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+fn main() {
+    uniffi::generate_scaffolding("./src/name-case.udl").unwrap();
+}

fixtures/name-case/src/lib.rs

@@ -0,0 +1,53 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+include!(concat!(env!("OUT_DIR"), "/name-case.uniffi.rs"));
+
+pub enum ENUMTest {
+    VARIANTOne,
+}
+
+pub enum AssociatedENUMTest {
+    VARIANTTest{ code: i16 },
+}
+
+#[derive(Debug, thiserror::Error)]
+pub enum ERRORTest {
+    #[error("Test")]
+    VARIANTOne,
+}
+
+#[derive(Debug, thiserror::Error)]
+pub enum AssociatedERRORTest {
+    #[error("Test")]
+    VARIANTTest{ code: i16 },
+}
+
+pub struct OBJECTTest {
+}
+
+impl OBJECTTest {
+    pub fn new() -> Self {
+        OBJECTTest{}
+    }
+
+    pub fn new_alternate() -> Self {
+        OBJECTTest{}
+    }
+
+    pub fn test(&self) {
+    }
+}
+
+pub struct RECORDTest {
+    test: i32,
+}
+
+pub fn test() {
+    let _ = ERRORTest::VARIANTOne;
+}
+
+pub trait CALLBACKTest {
+   fn test(&self);
+}

fixtures/name-case/src/name-case.udl

@@ -0,0 +1,35 @@
+namespace name_case {
+    void test();
+};
+
+enum ENUMTest { "VARIANTOne" };
+
+[Enum]
+interface AssociatedENUMTest {
+    VARIANTTest(i16 code);
+};
+
+[Error]
+enum ERRORTest { "VARIANTOne" };
+
+[Error]
+interface AssociatedERRORTest {
+    VARIANTTest(i16 code);
+};
+
+interface OBJECTTest {
+    constructor();
+
+    [Name="new_alternate"]
+    constructor();
+
+    void test();
+};
+
+dictionary RECORDTest {
+    i32 test;
+};
+
+callback interface CALLBACKTest {
+    void test();
+};

fixtures/src/lib.rs

@@ -16,7 +16,8 @@ mod uniffi_fixtures {
     uniffi_coverall::uniffi_reexport_scaffolding!();
     uniffi_external_types_lib::uniffi_reexport_scaffolding!();
 
-    uniffi_go_errors::uniffi_reexport_scaffolding!();
     uniffi_go_destroy::uniffi_reexport_scaffolding!();
+    uniffi_go_errors::uniffi_reexport_scaffolding!();
+    uniffi_go_name_case::uniffi_reexport_scaffolding!();
     uniffi_go_objects::uniffi_reexport_scaffolding!();
 }

test_bindings.sh

@@ -25,6 +25,7 @@ bindings 3rd-party/uniffi-rs/fixtures/coverall/src/coverall.udl
 bindings 3rd-party/uniffi-rs/fixtures/uniffi-fixture-time/src/chronological.udl
 bindings fixtures/destroy/src/destroy.udl
 bindings fixtures/errors/src/errors.udl
+bindings fixtures/name-case/src/name-case.udl
 bindings fixtures/objects/src/objects.udl
 
 pushd $BINDINGS_DIR/..