THRIFT-2689 struct names that differ only in capitalization of first character generate broken erlang code
Client: haskell
Patch: Alisdair Sullivan
This closes #239
Removed struct from NameConflictTest.thrift that causes problems for certain backends
diff --git a/test/NameConflictTest.thrift b/test/NameConflictTest.thrift
index 337be07..e5140dd 100644
--- a/test/NameConflictTest.thrift
+++ b/test/NameConflictTest.thrift
@@ -76,15 +76,6 @@
6: bool TheEdgeCase__
}
-struct theEdgeCase {
- 1: bool theEdgeCase
- 2: bool theEdgeCase_
- 3: bool theEdgeCase__
- 4: bool TheEdgeCase
- 5: bool TheEdgeCase_
- 6: bool TheEdgeCase__
-}
-
struct Tricky_ {
1: bool tricky
2: bool Tricky
diff --git a/test/erl/src/name_conflict_test.erl b/test/erl/src/name_conflict_test.erl
index 40c8204..a38eff0 100644
--- a/test/erl/src/name_conflict_test.erl
+++ b/test/erl/src/name_conflict_test.erl
@@ -80,17 +80,6 @@
'TheEdgeCase__'=null
}
)},
- {"theEdgeCase record", ?_assertMatch(
- {theEdgeCase, _, _, _, _, _, _},
- #theEdgeCase{
- theEdgeCase=null,
- theEdgeCase_=null,
- theEdgeCase__=null,
- 'TheEdgeCase'=null,
- 'TheEdgeCase_'=null,
- 'TheEdgeCase__'=null
- }
- )},
{"Tricky_ record", ?_assertMatch(
{'Tricky_', _, _},
#'Tricky_'{tricky=null,'Tricky'=null}
@@ -149,10 +138,6 @@
{struct, [{1, bool},{2, bool},{3, bool},{4, bool},{5, bool},{6, bool}]},
name_conflict_test_types:struct_info('TheEdgeCase')
)},
- {"theEdgeCase definition", ?_assertEqual(
- {struct, [{1, bool},{2, bool},{3, bool},{4, bool},{5, bool},{6, bool}]},
- name_conflict_test_types:struct_info(theEdgeCase)
- )},
{"Tricky_ definition", ?_assertEqual(
{struct, [{1, bool},{2, bool}]},
name_conflict_test_types:struct_info('Tricky_')
@@ -234,17 +219,6 @@
]},
name_conflict_test_types:struct_info_ext('TheEdgeCase')
)},
- {"TheEdgeCase extended definition", ?_assertEqual(
- {struct, [
- {1, undefined, bool, theEdgeCase, undefined},
- {2, undefined, bool, theEdgeCase_, undefined},
- {3, undefined, bool, theEdgeCase__, undefined},
- {4, undefined, bool, 'TheEdgeCase', undefined},
- {5, undefined, bool, 'TheEdgeCase_', undefined},
- {6, undefined, bool, 'TheEdgeCase__', undefined}
- ]},
- name_conflict_test_types:struct_info_ext(theEdgeCase)
- )},
{"Tricky_ extended definition", ?_assertEqual(
{struct, [
{1, undefined, bool, tricky, undefined},