ibm_db 3.0.5-x86-mingw32 → 4.0.0-x86-mingw32

Sign up to get free protection for your applications and to get access to all the features.
Files changed (586) hide show
  1. checksums.yaml +4 -4
  2. data/CHANGES +4 -0
  3. data/LICENSE +1 -1
  4. data/MANIFEST +14 -14
  5. data/ParameterizedQueries README +6 -6
  6. data/README +208 -225
  7. data/ext/Makefile.nt32 +181 -181
  8. data/ext/Makefile.nt32.191 +212 -212
  9. data/ext/extconf.rb +291 -291
  10. data/ext/ibm_db.c +11887 -11887
  11. data/ext/ruby_ibm_db.h +241 -241
  12. data/ext/ruby_ibm_db_cli.c +866 -866
  13. data/ext/ruby_ibm_db_cli.h +500 -500
  14. data/init.rb +41 -41
  15. data/lib/IBM_DB.rb +27 -27
  16. data/lib/active_record/connection_adapters/ibm_db_adapter.rb +3452 -3177
  17. data/lib/active_record/connection_adapters/ibmdb_adapter.rb +5 -2
  18. data/lib/active_record/vendor/db2-i5-zOS.yaml +328 -328
  19. data/lib/mswin32/ibm_db.rb +91 -123
  20. data/lib/mswin32/rb2x/i386/ibm_db.so +0 -0
  21. data/test/active_record/connection_adapters/fake_adapter.rb +49 -46
  22. data/test/assets/example.log +1 -1
  23. data/test/assets/test.txt +1 -1
  24. data/test/cases/adapter_test.rb +351 -276
  25. data/test/cases/adapters/mysql2/active_schema_test.rb +193 -0
  26. data/test/cases/adapters/mysql2/bind_parameter_test.rb +50 -0
  27. data/test/cases/adapters/mysql2/boolean_test.rb +100 -0
  28. data/test/cases/adapters/mysql2/case_sensitivity_test.rb +63 -0
  29. data/test/cases/adapters/mysql2/charset_collation_test.rb +54 -0
  30. data/test/cases/adapters/mysql2/connection_test.rb +210 -0
  31. data/test/cases/adapters/mysql2/datetime_precision_quoting_test.rb +45 -0
  32. data/test/cases/adapters/mysql2/enum_test.rb +26 -0
  33. data/test/cases/adapters/mysql2/explain_test.rb +21 -0
  34. data/test/cases/adapters/mysql2/json_test.rb +195 -0
  35. data/test/cases/adapters/mysql2/mysql2_adapter_test.rb +83 -0
  36. data/test/cases/adapters/mysql2/reserved_word_test.rb +152 -0
  37. data/test/cases/adapters/mysql2/schema_migrations_test.rb +59 -0
  38. data/test/cases/adapters/mysql2/schema_test.rb +126 -0
  39. data/test/cases/adapters/mysql2/sp_test.rb +36 -0
  40. data/test/cases/adapters/mysql2/sql_types_test.rb +14 -0
  41. data/test/cases/adapters/mysql2/table_options_test.rb +42 -0
  42. data/test/cases/adapters/mysql2/unsigned_type_test.rb +66 -0
  43. data/test/cases/adapters/postgresql/active_schema_test.rb +98 -0
  44. data/test/cases/adapters/postgresql/array_test.rb +339 -0
  45. data/test/cases/adapters/postgresql/bit_string_test.rb +82 -0
  46. data/test/cases/adapters/postgresql/bytea_test.rb +134 -0
  47. data/test/cases/adapters/postgresql/case_insensitive_test.rb +26 -0
  48. data/test/cases/adapters/postgresql/change_schema_test.rb +38 -0
  49. data/test/cases/adapters/postgresql/cidr_test.rb +25 -0
  50. data/test/cases/adapters/postgresql/citext_test.rb +78 -0
  51. data/test/cases/adapters/postgresql/collation_test.rb +53 -0
  52. data/test/cases/adapters/postgresql/composite_test.rb +132 -0
  53. data/test/cases/adapters/postgresql/connection_test.rb +257 -0
  54. data/test/cases/adapters/postgresql/datatype_test.rb +92 -0
  55. data/test/cases/adapters/postgresql/domain_test.rb +47 -0
  56. data/test/cases/adapters/postgresql/enum_test.rb +91 -0
  57. data/test/cases/adapters/postgresql/explain_test.rb +20 -0
  58. data/test/cases/adapters/postgresql/extension_migration_test.rb +63 -0
  59. data/test/cases/adapters/postgresql/full_text_test.rb +44 -0
  60. data/test/cases/adapters/postgresql/geometric_test.rb +378 -0
  61. data/test/cases/adapters/postgresql/hstore_test.rb +382 -0
  62. data/test/cases/adapters/postgresql/infinity_test.rb +69 -0
  63. data/test/cases/adapters/postgresql/integer_test.rb +25 -0
  64. data/test/cases/adapters/postgresql/json_test.rb +237 -0
  65. data/test/cases/adapters/postgresql/ltree_test.rb +53 -0
  66. data/test/cases/adapters/postgresql/money_test.rb +96 -0
  67. data/test/cases/adapters/postgresql/network_test.rb +94 -0
  68. data/test/cases/adapters/postgresql/numbers_test.rb +49 -0
  69. data/test/cases/adapters/postgresql/postgresql_adapter_test.rb +405 -0
  70. data/test/cases/adapters/postgresql/prepared_statements_test.rb +22 -0
  71. data/test/cases/adapters/postgresql/quoting_test.rb +44 -0
  72. data/test/cases/adapters/postgresql/range_test.rb +343 -0
  73. data/test/cases/adapters/postgresql/referential_integrity_test.rb +111 -0
  74. data/test/cases/adapters/postgresql/rename_table_test.rb +34 -0
  75. data/test/cases/adapters/postgresql/schema_authorization_test.rb +119 -0
  76. data/test/cases/adapters/postgresql/schema_test.rb +597 -0
  77. data/test/cases/adapters/postgresql/serial_test.rb +154 -0
  78. data/test/cases/adapters/postgresql/statement_pool_test.rb +41 -0
  79. data/test/cases/adapters/postgresql/timestamp_test.rb +90 -0
  80. data/test/cases/adapters/postgresql/type_lookup_test.rb +33 -0
  81. data/test/cases/adapters/postgresql/utils_test.rb +62 -0
  82. data/test/cases/adapters/postgresql/uuid_test.rb +294 -0
  83. data/test/cases/adapters/postgresql/xml_test.rb +54 -0
  84. data/test/cases/adapters/sqlite3/collation_test.rb +53 -0
  85. data/test/cases/adapters/sqlite3/copy_table_test.rb +98 -0
  86. data/test/cases/adapters/sqlite3/explain_test.rb +21 -0
  87. data/test/cases/adapters/sqlite3/quoting_test.rb +101 -0
  88. data/test/cases/adapters/sqlite3/sqlite3_adapter_test.rb +441 -0
  89. data/test/cases/adapters/sqlite3/sqlite3_create_folder_test.rb +24 -0
  90. data/test/cases/adapters/sqlite3/statement_pool_test.rb +20 -0
  91. data/test/cases/aggregations_test.rb +168 -158
  92. data/test/cases/ar_schema_test.rb +146 -161
  93. data/test/cases/associations/association_scope_test.rb +16 -21
  94. data/test/cases/associations/belongs_to_associations_test.rb +1141 -1029
  95. data/test/cases/associations/bidirectional_destroy_dependencies_test.rb +41 -0
  96. data/test/cases/associations/callbacks_test.rb +190 -192
  97. data/test/cases/associations/cascaded_eager_loading_test.rb +188 -188
  98. data/test/cases/associations/eager_load_includes_full_sti_class_test.rb +36 -36
  99. data/test/cases/associations/eager_load_nested_include_test.rb +126 -128
  100. data/test/cases/associations/eager_singularization_test.rb +148 -148
  101. data/test/cases/associations/eager_test.rb +1514 -1429
  102. data/test/cases/associations/extension_test.rb +87 -82
  103. data/test/cases/associations/has_and_belongs_to_many_associations_test.rb +1004 -972
  104. data/test/cases/associations/has_many_associations_test.rb +2501 -2182
  105. data/test/cases/associations/has_many_through_associations_test.rb +1271 -1204
  106. data/test/cases/associations/has_one_associations_test.rb +707 -610
  107. data/test/cases/associations/has_one_through_associations_test.rb +383 -380
  108. data/test/cases/associations/inner_join_association_test.rb +139 -139
  109. data/test/cases/associations/inverse_associations_test.rb +733 -706
  110. data/test/cases/associations/join_model_test.rb +777 -754
  111. data/test/cases/associations/left_outer_join_association_test.rb +88 -0
  112. data/test/cases/associations/nested_through_associations_test.rb +579 -579
  113. data/test/cases/associations/required_test.rb +102 -82
  114. data/test/cases/associations_test.rb +385 -380
  115. data/test/cases/attribute_decorators_test.rb +125 -125
  116. data/test/cases/attribute_methods/read_test.rb +60 -60
  117. data/test/cases/attribute_methods_test.rb +1009 -952
  118. data/test/cases/attribute_set_test.rb +270 -210
  119. data/test/cases/attribute_test.rb +246 -180
  120. data/test/cases/attributes_test.rb +253 -136
  121. data/test/cases/autosave_association_test.rb +1708 -1595
  122. data/test/cases/base_test.rb +1713 -1664
  123. data/test/cases/batches_test.rb +489 -212
  124. data/test/cases/binary_test.rb +44 -52
  125. data/test/cases/bind_parameter_test.rb +110 -100
  126. data/test/cases/cache_key_test.rb +25 -0
  127. data/test/cases/calculations_test.rb +798 -646
  128. data/test/cases/callbacks_test.rb +636 -543
  129. data/test/cases/clone_test.rb +40 -40
  130. data/test/cases/coders/json_test.rb +15 -0
  131. data/test/cases/coders/yaml_column_test.rb +63 -63
  132. data/test/cases/collection_cache_key_test.rb +115 -0
  133. data/test/cases/column_alias_test.rb +17 -17
  134. data/test/cases/column_definition_test.rb +92 -123
  135. data/test/cases/comment_test.rb +143 -0
  136. data/test/cases/connection_adapters/adapter_leasing_test.rb +56 -54
  137. data/test/cases/connection_adapters/connection_handler_test.rb +160 -53
  138. data/test/cases/connection_adapters/connection_specification_test.rb +12 -12
  139. data/test/cases/connection_adapters/merge_and_resolve_default_url_config_test.rb +255 -293
  140. data/test/cases/connection_adapters/mysql_type_lookup_test.rb +69 -65
  141. data/test/cases/connection_adapters/quoting_test.rb +13 -13
  142. data/test/cases/connection_adapters/schema_cache_test.rb +61 -56
  143. data/test/cases/connection_adapters/type_lookup_test.rb +118 -110
  144. data/test/cases/connection_management_test.rb +112 -122
  145. data/test/cases/connection_pool_test.rb +521 -346
  146. data/test/cases/connection_specification/resolver_test.rb +131 -116
  147. data/test/cases/core_test.rb +112 -112
  148. data/test/cases/counter_cache_test.rb +214 -209
  149. data/test/cases/custom_locking_test.rb +17 -17
  150. data/test/cases/database_statements_test.rb +34 -19
  151. data/test/cases/{invalid_date_test.rb → date_test.rb} +44 -32
  152. data/test/cases/date_time_precision_test.rb +106 -0
  153. data/test/cases/date_time_test.rb +61 -61
  154. data/test/cases/defaults_test.rb +218 -223
  155. data/test/cases/dirty_test.rb +763 -785
  156. data/test/cases/disconnected_test.rb +30 -28
  157. data/test/cases/dup_test.rb +157 -157
  158. data/test/cases/enum_test.rb +444 -290
  159. data/test/cases/errors_test.rb +16 -0
  160. data/test/cases/explain_subscriber_test.rb +64 -64
  161. data/test/cases/explain_test.rb +87 -76
  162. data/test/cases/finder_respond_to_test.rb +60 -60
  163. data/test/cases/finder_test.rb +1294 -1169
  164. data/test/cases/fixture_set/file_test.rb +156 -138
  165. data/test/cases/fixtures_test.rb +988 -908
  166. data/test/cases/forbidden_attributes_protection_test.rb +165 -99
  167. data/test/cases/habtm_destroy_order_test.rb +61 -61
  168. data/test/cases/helper.rb +204 -210
  169. data/test/cases/hot_compatibility_test.rb +142 -54
  170. data/test/cases/i18n_test.rb +45 -45
  171. data/test/cases/inheritance_test.rb +606 -375
  172. data/test/cases/integration_test.rb +155 -139
  173. data/test/cases/invalid_connection_test.rb +24 -22
  174. data/test/cases/invertible_migration_test.rb +387 -295
  175. data/test/cases/json_serialization_test.rb +311 -302
  176. data/test/cases/locking_test.rb +493 -477
  177. data/test/cases/log_subscriber_test.rb +225 -136
  178. data/test/cases/migration/change_schema_test.rb +458 -512
  179. data/test/cases/migration/change_table_test.rb +256 -224
  180. data/test/cases/migration/column_attributes_test.rb +176 -192
  181. data/test/cases/migration/column_positioning_test.rb +56 -56
  182. data/test/cases/migration/columns_test.rb +310 -304
  183. data/test/cases/migration/command_recorder_test.rb +350 -305
  184. data/test/cases/migration/compatibility_test.rb +118 -0
  185. data/test/cases/migration/create_join_table_test.rb +157 -148
  186. data/test/cases/migration/foreign_key_test.rb +360 -328
  187. data/test/cases/migration/helper.rb +39 -39
  188. data/test/cases/migration/index_test.rb +218 -216
  189. data/test/cases/migration/logger_test.rb +36 -36
  190. data/test/cases/migration/pending_migrations_test.rb +52 -53
  191. data/test/cases/migration/references_foreign_key_test.rb +216 -169
  192. data/test/cases/migration/references_index_test.rb +101 -101
  193. data/test/cases/migration/references_statements_test.rb +136 -116
  194. data/test/cases/migration/rename_table_test.rb +93 -93
  195. data/test/cases/migration_test.rb +1157 -959
  196. data/test/cases/migrator_test.rb +470 -388
  197. data/test/cases/mixin_test.rb +68 -70
  198. data/test/cases/modules_test.rb +172 -173
  199. data/test/cases/multiparameter_attributes_test.rb +372 -350
  200. data/test/cases/multiple_db_test.rb +122 -115
  201. data/test/cases/nested_attributes_test.rb +1098 -1070
  202. data/test/cases/nested_attributes_with_callbacks_test.rb +144 -144
  203. data/test/cases/persistence_test.rb +1001 -909
  204. data/test/cases/pooled_connections_test.rb +81 -81
  205. data/test/cases/primary_keys_test.rb +376 -237
  206. data/test/cases/query_cache_test.rb +446 -326
  207. data/test/cases/quoting_test.rb +202 -156
  208. data/test/cases/readonly_test.rb +119 -118
  209. data/test/cases/reaper_test.rb +85 -85
  210. data/test/cases/reflection_test.rb +509 -463
  211. data/test/cases/relation/delegation_test.rb +63 -68
  212. data/test/cases/relation/merging_test.rb +157 -161
  213. data/test/cases/relation/mutation_test.rb +183 -165
  214. data/test/cases/relation/or_test.rb +92 -0
  215. data/test/cases/relation/predicate_builder_test.rb +16 -14
  216. data/test/cases/relation/record_fetch_warning_test.rb +40 -0
  217. data/test/cases/relation/where_chain_test.rb +105 -181
  218. data/test/cases/relation/where_clause_test.rb +182 -0
  219. data/test/cases/relation/where_test.rb +322 -300
  220. data/test/cases/relation_test.rb +328 -319
  221. data/test/cases/relations_test.rb +2026 -1815
  222. data/test/cases/reload_models_test.rb +22 -22
  223. data/test/cases/result_test.rb +90 -80
  224. data/test/cases/sanitize_test.rb +176 -83
  225. data/test/cases/schema_dumper_test.rb +457 -463
  226. data/test/cases/schema_loading_test.rb +52 -0
  227. data/test/cases/scoping/default_scoping_test.rb +528 -454
  228. data/test/cases/scoping/named_scoping_test.rb +561 -524
  229. data/test/cases/scoping/relation_scoping_test.rb +400 -357
  230. data/test/cases/secure_token_test.rb +32 -0
  231. data/test/cases/serialization_test.rb +104 -104
  232. data/test/cases/serialized_attribute_test.rb +364 -277
  233. data/test/cases/statement_cache_test.rb +136 -98
  234. data/test/cases/store_test.rb +195 -194
  235. data/test/cases/suppressor_test.rb +63 -0
  236. data/test/cases/tasks/database_tasks_test.rb +462 -398
  237. data/test/cases/tasks/mysql_rake_test.rb +345 -324
  238. data/test/cases/tasks/postgresql_rake_test.rb +304 -250
  239. data/test/cases/tasks/sqlite_rake_test.rb +220 -193
  240. data/test/cases/test_case.rb +131 -123
  241. data/test/cases/test_fixtures_test.rb +36 -0
  242. data/test/cases/time_precision_test.rb +102 -0
  243. data/test/cases/timestamp_test.rb +501 -467
  244. data/test/cases/touch_later_test.rb +121 -0
  245. data/test/cases/transaction_callbacks_test.rb +518 -452
  246. data/test/cases/transaction_isolation_test.rb +106 -106
  247. data/test/cases/transactions_test.rb +834 -817
  248. data/test/cases/type/adapter_specific_registry_test.rb +133 -0
  249. data/test/cases/type/date_time_test.rb +14 -0
  250. data/test/cases/type/integer_test.rb +27 -121
  251. data/test/cases/type/string_test.rb +22 -36
  252. data/test/cases/type/type_map_test.rb +177 -177
  253. data/test/cases/type_test.rb +39 -0
  254. data/test/cases/types_test.rb +24 -141
  255. data/test/cases/unconnected_test.rb +33 -33
  256. data/test/cases/validations/absence_validation_test.rb +73 -0
  257. data/test/cases/validations/association_validation_test.rb +97 -86
  258. data/test/cases/validations/i18n_generate_message_validation_test.rb +84 -84
  259. data/test/cases/validations/i18n_validation_test.rb +86 -90
  260. data/test/cases/validations/length_validation_test.rb +79 -47
  261. data/test/cases/validations/presence_validation_test.rb +103 -68
  262. data/test/cases/validations/uniqueness_validation_test.rb +548 -457
  263. data/test/cases/validations_repair_helper.rb +19 -23
  264. data/test/cases/validations_test.rb +194 -165
  265. data/test/cases/view_test.rb +216 -119
  266. data/test/cases/yaml_serialization_test.rb +121 -126
  267. data/test/config.example.yml +97 -0
  268. data/test/config.rb +5 -5
  269. data/test/fixtures/accounts.yml +29 -29
  270. data/test/fixtures/admin/accounts.yml +2 -2
  271. data/test/fixtures/admin/users.yml +10 -10
  272. data/test/fixtures/author_addresses.original +11 -0
  273. data/test/fixtures/author_addresses.yml +17 -17
  274. data/test/fixtures/author_favorites.yml +3 -3
  275. data/test/fixtures/authors.original +17 -0
  276. data/test/fixtures/authors.yml +23 -23
  277. data/test/fixtures/bad_posts.yml +9 -0
  278. data/test/fixtures/binaries.yml +133 -133
  279. data/test/fixtures/books.yml +31 -11
  280. data/test/fixtures/bulbs.yml +5 -5
  281. data/test/fixtures/cars.yml +9 -9
  282. data/test/fixtures/categories.yml +19 -19
  283. data/test/fixtures/categories/special_categories.yml +9 -9
  284. data/test/fixtures/categories/subsubdir/arbitrary_filename.yml +4 -4
  285. data/test/fixtures/categories_ordered.yml +7 -7
  286. data/test/fixtures/categories_posts.yml +31 -31
  287. data/test/fixtures/categorizations.yml +23 -23
  288. data/test/fixtures/clubs.yml +8 -8
  289. data/test/fixtures/collections.yml +3 -3
  290. data/test/fixtures/colleges.yml +3 -3
  291. data/test/fixtures/comments.yml +65 -65
  292. data/test/fixtures/companies.yml +67 -67
  293. data/test/fixtures/computers.yml +10 -10
  294. data/test/fixtures/content.yml +3 -0
  295. data/test/fixtures/content_positions.yml +3 -0
  296. data/test/fixtures/courses.yml +8 -8
  297. data/test/fixtures/customers.yml +25 -25
  298. data/test/fixtures/dashboards.yml +6 -6
  299. data/test/fixtures/dead_parrots.yml +5 -0
  300. data/test/fixtures/developers.yml +22 -22
  301. data/test/fixtures/developers_projects.yml +16 -16
  302. data/test/fixtures/dog_lovers.yml +7 -7
  303. data/test/fixtures/dogs.yml +4 -4
  304. data/test/fixtures/doubloons.yml +3 -3
  305. data/test/fixtures/edges.yml +5 -5
  306. data/test/fixtures/entrants.yml +14 -14
  307. data/test/fixtures/essays.yml +6 -6
  308. data/test/fixtures/faces.yml +11 -11
  309. data/test/fixtures/fk_test_has_fk.yml +3 -3
  310. data/test/fixtures/fk_test_has_pk.yml +1 -1
  311. data/test/fixtures/friendships.yml +4 -4
  312. data/test/fixtures/funny_jokes.yml +10 -10
  313. data/test/fixtures/interests.yml +33 -33
  314. data/test/fixtures/items.yml +3 -3
  315. data/test/fixtures/jobs.yml +7 -7
  316. data/test/fixtures/legacy_things.yml +3 -3
  317. data/test/fixtures/live_parrots.yml +4 -0
  318. data/test/fixtures/mateys.yml +4 -4
  319. data/test/fixtures/member_details.yml +8 -8
  320. data/test/fixtures/member_types.yml +6 -6
  321. data/test/fixtures/members.yml +11 -11
  322. data/test/fixtures/memberships.yml +34 -34
  323. data/test/fixtures/men.yml +5 -5
  324. data/test/fixtures/minimalistics.yml +2 -2
  325. data/test/fixtures/minivans.yml +5 -5
  326. data/test/fixtures/mixed_case_monkeys.yml +6 -6
  327. data/test/fixtures/mixins.yml +29 -29
  328. data/test/fixtures/movies.yml +7 -7
  329. data/test/fixtures/naked/yml/accounts.yml +1 -1
  330. data/test/fixtures/naked/yml/companies.yml +1 -1
  331. data/test/fixtures/naked/yml/courses.yml +1 -1
  332. data/test/fixtures/naked/yml/parrots.yml +2 -0
  333. data/test/fixtures/naked/yml/trees.yml +3 -0
  334. data/test/fixtures/nodes.yml +29 -0
  335. data/test/fixtures/organizations.yml +5 -5
  336. data/test/fixtures/other_comments.yml +6 -0
  337. data/test/fixtures/other_dogs.yml +2 -0
  338. data/test/fixtures/other_posts.yml +7 -0
  339. data/test/fixtures/other_topics.yml +42 -42
  340. data/test/fixtures/owners.yml +9 -9
  341. data/test/fixtures/parrots.yml +27 -27
  342. data/test/fixtures/parrots_pirates.yml +7 -7
  343. data/test/fixtures/people.yml +24 -24
  344. data/test/fixtures/peoples_treasures.yml +3 -3
  345. data/test/fixtures/pets.yml +19 -19
  346. data/test/fixtures/pirates.yml +15 -12
  347. data/test/fixtures/posts.yml +80 -80
  348. data/test/fixtures/price_estimates.yml +16 -7
  349. data/test/fixtures/products.yml +4 -4
  350. data/test/fixtures/projects.yml +7 -7
  351. data/test/fixtures/ratings.yml +14 -14
  352. data/test/fixtures/readers.yml +11 -11
  353. data/test/fixtures/references.yml +17 -17
  354. data/test/fixtures/reserved_words/distinct.yml +5 -5
  355. data/test/fixtures/reserved_words/distinct_select.yml +11 -11
  356. data/test/fixtures/reserved_words/group.yml +14 -14
  357. data/test/fixtures/reserved_words/select.yml +8 -8
  358. data/test/fixtures/reserved_words/values.yml +7 -7
  359. data/test/fixtures/ships.yml +6 -6
  360. data/test/fixtures/speedometers.yml +8 -8
  361. data/test/fixtures/sponsors.yml +12 -12
  362. data/test/fixtures/string_key_objects.yml +7 -7
  363. data/test/fixtures/subscribers.yml +10 -10
  364. data/test/fixtures/subscriptions.yml +12 -12
  365. data/test/fixtures/taggings.yml +78 -78
  366. data/test/fixtures/tags.yml +11 -11
  367. data/test/fixtures/tasks.yml +7 -7
  368. data/test/fixtures/teapots.yml +3 -3
  369. data/test/fixtures/to_be_linked/accounts.yml +2 -2
  370. data/test/fixtures/to_be_linked/users.yml +10 -10
  371. data/test/fixtures/topics.yml +49 -49
  372. data/test/fixtures/toys.yml +14 -14
  373. data/test/fixtures/traffic_lights.yml +9 -9
  374. data/test/fixtures/treasures.yml +10 -10
  375. data/test/fixtures/trees.yml +3 -0
  376. data/test/fixtures/uuid_children.yml +3 -3
  377. data/test/fixtures/uuid_parents.yml +2 -2
  378. data/test/fixtures/variants.yml +4 -4
  379. data/test/fixtures/vegetables.yml +19 -19
  380. data/test/fixtures/vertices.yml +3 -3
  381. data/test/fixtures/warehouse_things.yml +2 -2
  382. data/test/fixtures/zines.yml +5 -5
  383. data/test/migrations/10_urban/9_add_expressions.rb +11 -11
  384. data/test/migrations/decimal/1_give_me_big_numbers.rb +15 -15
  385. data/test/migrations/magic/1_currencies_have_symbols.rb +12 -12
  386. data/test/migrations/missing/1000_people_have_middle_names.rb +9 -9
  387. data/test/migrations/missing/1_people_have_last_names.rb +9 -9
  388. data/test/migrations/missing/3_we_need_reminders.rb +12 -12
  389. data/test/migrations/missing/4_innocent_jointable.rb +12 -12
  390. data/test/migrations/rename/1_we_need_things.rb +11 -11
  391. data/test/migrations/rename/2_rename_things.rb +9 -9
  392. data/test/migrations/to_copy/1_people_have_hobbies.rb +9 -9
  393. data/test/migrations/to_copy/2_people_have_descriptions.rb +9 -9
  394. data/test/migrations/to_copy2/1_create_articles.rb +7 -7
  395. data/test/migrations/to_copy2/2_create_comments.rb +7 -7
  396. data/test/migrations/to_copy_with_name_collision/1_people_have_hobbies.rb +9 -9
  397. data/test/migrations/to_copy_with_timestamps/20090101010101_people_have_hobbies.rb +9 -9
  398. data/test/migrations/to_copy_with_timestamps/20090101010202_people_have_descriptions.rb +9 -9
  399. data/test/migrations/to_copy_with_timestamps2/20090101010101_create_articles.rb +7 -7
  400. data/test/migrations/to_copy_with_timestamps2/20090101010202_create_comments.rb +7 -7
  401. data/test/migrations/valid/1_valid_people_have_last_names.rb +9 -9
  402. data/test/migrations/valid/2_we_need_reminders.rb +12 -12
  403. data/test/migrations/valid/3_innocent_jointable.rb +12 -12
  404. data/test/migrations/valid_with_subdirectories/1_valid_people_have_last_names.rb +9 -9
  405. data/test/migrations/valid_with_subdirectories/sub/2_we_need_reminders.rb +12 -12
  406. data/test/migrations/valid_with_subdirectories/sub1/3_innocent_jointable.rb +12 -12
  407. data/test/migrations/valid_with_timestamps/20100101010101_valid_with_timestamps_people_have_last_names.rb +9 -9
  408. data/test/migrations/valid_with_timestamps/20100201010101_valid_with_timestamps_we_need_reminders.rb +12 -12
  409. data/test/migrations/valid_with_timestamps/20100301010101_valid_with_timestamps_innocent_jointable.rb +12 -12
  410. data/test/migrations/version_check/20131219224947_migration_version_check.rb +8 -8
  411. data/test/models/admin.rb +5 -5
  412. data/test/models/admin/account.rb +3 -3
  413. data/test/models/admin/randomly_named_c1.rb +6 -2
  414. data/test/models/admin/user.rb +40 -40
  415. data/test/models/aircraft.rb +5 -4
  416. data/test/models/arunit2_model.rb +3 -3
  417. data/test/models/author.rb +209 -212
  418. data/test/models/auto_id.rb +4 -4
  419. data/test/models/autoloadable/extra_firm.rb +2 -2
  420. data/test/models/binary.rb +2 -2
  421. data/test/models/bird.rb +12 -12
  422. data/test/models/book.rb +23 -18
  423. data/test/models/boolean.rb +2 -2
  424. data/test/models/bulb.rb +52 -51
  425. data/test/models/cake_designer.rb +3 -3
  426. data/test/models/car.rb +29 -26
  427. data/test/models/carrier.rb +2 -2
  428. data/test/models/cat.rb +10 -0
  429. data/test/models/categorization.rb +19 -19
  430. data/test/models/category.rb +35 -35
  431. data/test/models/chef.rb +8 -7
  432. data/test/models/citation.rb +3 -3
  433. data/test/models/club.rb +25 -23
  434. data/test/models/college.rb +10 -10
  435. data/test/models/column.rb +3 -3
  436. data/test/models/column_name.rb +3 -3
  437. data/test/models/comment.rb +76 -64
  438. data/test/models/company.rb +230 -228
  439. data/test/models/company_in_module.rb +98 -98
  440. data/test/models/computer.rb +3 -3
  441. data/test/models/contact.rb +41 -41
  442. data/test/models/content.rb +40 -0
  443. data/test/models/contract.rb +20 -20
  444. data/test/models/country.rb +7 -7
  445. data/test/models/course.rb +6 -6
  446. data/test/models/customer.rb +83 -77
  447. data/test/models/customer_carrier.rb +14 -14
  448. data/test/models/dashboard.rb +3 -3
  449. data/test/models/default.rb +2 -2
  450. data/test/models/department.rb +4 -4
  451. data/test/models/developer.rb +274 -255
  452. data/test/models/dog.rb +5 -5
  453. data/test/models/dog_lover.rb +5 -5
  454. data/test/models/doubloon.rb +12 -12
  455. data/test/models/drink_designer.rb +3 -3
  456. data/test/models/edge.rb +5 -5
  457. data/test/models/electron.rb +5 -5
  458. data/test/models/engine.rb +4 -4
  459. data/test/models/entrant.rb +3 -3
  460. data/test/models/essay.rb +5 -5
  461. data/test/models/event.rb +3 -3
  462. data/test/models/eye.rb +37 -37
  463. data/test/models/face.rb +9 -9
  464. data/test/models/friendship.rb +6 -6
  465. data/test/models/guid.rb +2 -2
  466. data/test/models/guitar.rb +4 -0
  467. data/test/models/hotel.rb +11 -9
  468. data/test/models/image.rb +3 -3
  469. data/test/models/interest.rb +5 -5
  470. data/test/models/invoice.rb +4 -4
  471. data/test/models/item.rb +7 -7
  472. data/test/models/job.rb +7 -7
  473. data/test/models/joke.rb +7 -7
  474. data/test/models/keyboard.rb +3 -3
  475. data/test/models/legacy_thing.rb +3 -3
  476. data/test/models/lesson.rb +11 -11
  477. data/test/models/line_item.rb +3 -3
  478. data/test/models/liquid.rb +4 -4
  479. data/test/models/man.rb +11 -11
  480. data/test/models/matey.rb +4 -4
  481. data/test/models/member.rb +42 -41
  482. data/test/models/member_detail.rb +8 -7
  483. data/test/models/member_type.rb +3 -3
  484. data/test/models/membership.rb +35 -35
  485. data/test/models/mentor.rb +3 -0
  486. data/test/models/minimalistic.rb +2 -2
  487. data/test/models/minivan.rb +9 -9
  488. data/test/models/mixed_case_monkey.rb +3 -3
  489. data/test/models/mocktail_designer.rb +2 -0
  490. data/test/models/molecule.rb +6 -6
  491. data/test/models/movie.rb +5 -5
  492. data/test/models/node.rb +5 -0
  493. data/test/models/non_primary_key.rb +2 -0
  494. data/test/models/notification.rb +3 -0
  495. data/test/models/order.rb +4 -4
  496. data/test/models/organization.rb +14 -14
  497. data/test/models/other_dog.rb +5 -0
  498. data/test/models/owner.rb +37 -34
  499. data/test/models/parrot.rb +28 -29
  500. data/test/models/person.rb +142 -143
  501. data/test/models/personal_legacy_thing.rb +4 -4
  502. data/test/models/pet.rb +18 -15
  503. data/test/models/pet_treasure.rb +6 -0
  504. data/test/models/pirate.rb +92 -92
  505. data/test/models/possession.rb +3 -3
  506. data/test/models/post.rb +273 -264
  507. data/test/models/price_estimate.rb +4 -4
  508. data/test/models/professor.rb +5 -5
  509. data/test/models/project.rb +40 -31
  510. data/test/models/publisher.rb +2 -2
  511. data/test/models/publisher/article.rb +4 -4
  512. data/test/models/publisher/magazine.rb +3 -3
  513. data/test/models/randomly_named_c1.rb +1 -1
  514. data/test/models/rating.rb +4 -4
  515. data/test/models/reader.rb +23 -23
  516. data/test/models/recipe.rb +3 -0
  517. data/test/models/record.rb +2 -2
  518. data/test/models/reference.rb +22 -22
  519. data/test/models/reply.rb +61 -61
  520. data/test/models/ship.rb +39 -33
  521. data/test/models/ship_part.rb +8 -8
  522. data/test/models/shop.rb +17 -17
  523. data/test/models/shop_account.rb +6 -6
  524. data/test/models/speedometer.rb +6 -6
  525. data/test/models/sponsor.rb +7 -7
  526. data/test/models/string_key_object.rb +3 -3
  527. data/test/models/student.rb +4 -4
  528. data/test/models/subject.rb +16 -16
  529. data/test/models/subscriber.rb +8 -8
  530. data/test/models/subscription.rb +4 -4
  531. data/test/models/tag.rb +13 -7
  532. data/test/models/tagging.rb +13 -13
  533. data/test/models/task.rb +5 -5
  534. data/test/models/topic.rb +118 -124
  535. data/test/models/toy.rb +6 -6
  536. data/test/models/traffic_light.rb +4 -4
  537. data/test/models/treasure.rb +14 -14
  538. data/test/models/treaty.rb +7 -7
  539. data/test/models/tree.rb +3 -0
  540. data/test/models/tuning_peg.rb +4 -0
  541. data/test/models/tyre.rb +11 -11
  542. data/test/models/user.rb +14 -0
  543. data/test/models/uuid_child.rb +3 -3
  544. data/test/models/uuid_item.rb +6 -0
  545. data/test/models/uuid_parent.rb +3 -3
  546. data/test/models/vegetables.rb +24 -24
  547. data/test/models/vehicle.rb +6 -6
  548. data/test/models/vertex.rb +9 -9
  549. data/test/models/warehouse_thing.rb +5 -5
  550. data/test/models/wheel.rb +3 -3
  551. data/test/models/without_table.rb +3 -3
  552. data/test/models/zine.rb +3 -3
  553. data/test/schema/mysql2_specific_schema.rb +68 -58
  554. data/test/schema/oracle_specific_schema.rb +40 -43
  555. data/test/schema/postgresql_specific_schema.rb +114 -202
  556. data/test/schema/schema.rb +1057 -952
  557. data/test/schema/schema.rb.original +1057 -0
  558. data/test/schema/sqlite_specific_schema.rb +18 -22
  559. data/test/support/config.rb +43 -43
  560. data/test/support/connection.rb +23 -22
  561. data/test/support/connection_helper.rb +14 -14
  562. data/test/support/ddl_helper.rb +8 -8
  563. data/test/support/schema_dumping_helper.rb +20 -20
  564. data/test/support/yaml_compatibility_fixtures/rails_4_1.yml +22 -0
  565. data/test/support/yaml_compatibility_fixtures/rails_4_2_0.yml +182 -0
  566. metadata +129 -28
  567. data/lib/mswin32/rb19x/ibm_db.so +0 -0
  568. data/lib/mswin32/rb21x/i386/ibm_db.so +0 -0
  569. data/lib/mswin32/rb22x/i386/ibm_db.so +0 -0
  570. data/lib/mswin32/rb23x/i386/ibm_db.so +0 -0
  571. data/test/cases/associations/deprecated_counter_cache_on_has_many_through_test.rb +0 -26
  572. data/test/cases/attribute_methods/serialization_test.rb +0 -29
  573. data/test/cases/migration/change_schema_test - Copy.rb +0 -448
  574. data/test/cases/migration/foreign_key_test - Changed.rb +0 -325
  575. data/test/cases/migration/table_and_index_test.rb +0 -24
  576. data/test/cases/relation/where_test2.rb +0 -36
  577. data/test/cases/type/decimal_test.rb +0 -56
  578. data/test/cases/type/unsigned_integer_test.rb +0 -18
  579. data/test/cases/xml_serialization_test.rb +0 -457
  580. data/test/connections/native_ibm_db/connection.rb +0 -44
  581. data/test/fixtures/naked/csv/accounts.csv +0 -1
  582. data/test/schema/i5/ibm_db_specific_schema.rb +0 -137
  583. data/test/schema/ids/ibm_db_specific_schema.rb +0 -140
  584. data/test/schema/luw/ibm_db_specific_schema.rb +0 -137
  585. data/test/schema/mysql_specific_schema.rb +0 -70
  586. data/test/schema/zOS/ibm_db_specific_schema.rb +0 -208
@@ -1,139 +1,139 @@
1
- require "cases/helper"
2
- require 'models/post'
3
- require 'models/comment'
4
- require 'models/author'
5
- require 'models/essay'
6
- require 'models/category'
7
- require 'models/categorization'
8
- require 'models/person'
9
- require 'models/tagging'
10
- require 'models/tag'
11
-
12
- class InnerJoinAssociationTest < ActiveRecord::TestCase
13
- fixtures :authors, :essays, :posts, :comments, :categories, :categories_posts, :categorizations,
14
- :taggings, :tags, :author_addresses
15
-
16
- def test_construct_finder_sql_applies_aliases_tables_on_association_conditions
17
- result = Author.joins(:thinking_posts, :welcome_posts).to_a
18
- assert_equal authors(:david), result.first
19
- end
20
-
21
- def test_construct_finder_sql_does_not_table_name_collide_on_duplicate_associations
22
- assert_nothing_raised do
23
- sql = Person.joins(:agents => {:agents => :agents}).joins(:agents => {:agents => {:primary_contact => :agents}}).to_sql
24
- assert_match(/agents_people_4/i, sql)
25
- end
26
- end
27
-
28
- def test_construct_finder_sql_ignores_empty_joins_hash
29
- sql = Author.joins({}).to_sql
30
- assert_no_match(/JOIN/i, sql)
31
- end
32
-
33
- def test_construct_finder_sql_ignores_empty_joins_array
34
- sql = Author.joins([]).to_sql
35
- assert_no_match(/JOIN/i, sql)
36
- end
37
-
38
- def test_join_conditions_added_to_join_clause
39
- sql = Author.joins(:essays).to_sql
40
- assert_match(/writer_type.*?=.*?Author/i, sql)
41
- assert_no_match(/WHERE/i, sql)
42
- end
43
-
44
- def test_join_association_conditions_support_string_and_arel_expressions
45
- assert_equal 0, Author.joins(:welcome_posts_with_one_comment).count
46
- assert_equal 1, Author.joins(:welcome_posts_with_comments).count
47
- end
48
-
49
- def test_join_conditions_allow_nil_associations
50
- authors = Author.includes(:essays).where(:essays => {:id => nil})
51
- assert_equal 2, authors.count
52
- end
53
-
54
- def test_find_with_implicit_inner_joins_without_select_does_not_imply_readonly
55
- authors = Author.joins(:posts)
56
- assert_not authors.empty?, "expected authors to be non-empty"
57
- assert authors.none? {|a| a.readonly? }, "expected no authors to be readonly"
58
- end
59
-
60
- def test_find_with_implicit_inner_joins_honors_readonly_with_select
61
- authors = Author.joins(:posts).select('authors.*').to_a
62
- assert !authors.empty?, "expected authors to be non-empty"
63
- assert authors.all? {|a| !a.readonly? }, "expected no authors to be readonly"
64
- end
65
-
66
- def test_find_with_implicit_inner_joins_honors_readonly_false
67
- authors = Author.joins(:posts).readonly(false).to_a
68
- assert !authors.empty?, "expected authors to be non-empty"
69
- assert authors.all? {|a| !a.readonly? }, "expected no authors to be readonly"
70
- end
71
-
72
- def test_find_with_implicit_inner_joins_does_not_set_associations
73
- authors = Author.joins(:posts).select('authors.*').to_a
74
- assert !authors.empty?, "expected authors to be non-empty"
75
- assert authors.all? { |a| !a.instance_variable_defined?(:@posts) }, "expected no authors to have the @posts association loaded"
76
- end
77
-
78
- def test_count_honors_implicit_inner_joins
79
- real_count = Author.all.to_a.sum{|a| a.posts.count }
80
- assert_equal real_count, Author.joins(:posts).count, "plain inner join count should match the number of referenced posts records"
81
- end
82
-
83
- def test_calculate_honors_implicit_inner_joins
84
- real_count = Author.all.to_a.sum{|a| a.posts.count }
85
- assert_equal real_count, Author.joins(:posts).calculate(:count, 'authors.id'), "plain inner join count should match the number of referenced posts records"
86
- end
87
-
88
- def test_calculate_honors_implicit_inner_joins_and_distinct_and_conditions
89
- real_count = Author.all.to_a.select {|a| a.posts.any? {|p| p.title =~ /^Welcome/} }.length
90
- authors_with_welcoming_post_titles = Author.all.merge!(joins: :posts, where: "posts.title like 'Welcome%'").distinct.calculate(:count, 'authors.id')
91
- assert_equal real_count, authors_with_welcoming_post_titles, "inner join and conditions should have only returned authors posting titles starting with 'Welcome'"
92
- end
93
-
94
- def test_find_with_sti_join
95
- scope = Post.joins(:special_comments).where(:id => posts(:sti_comments).id)
96
-
97
- # The join should match SpecialComment and its subclasses only
98
- assert scope.where("comments.type" => "Comment").empty?
99
- assert !scope.where("comments.type" => "SpecialComment").empty?
100
- assert !scope.where("comments.type" => "SubSpecialComment").empty?
101
- end
102
-
103
- def test_find_with_conditions_on_reflection
104
- assert !posts(:welcome).comments.empty?
105
- assert Post.joins(:nonexistant_comments).where(:id => posts(:welcome).id).empty? # [sic!]
106
- end
107
-
108
- def test_find_with_conditions_on_through_reflection
109
- assert !posts(:welcome).tags.empty?
110
- assert Post.joins(:misc_tags).where(:id => posts(:welcome).id).empty?
111
- end
112
-
113
- test "the default scope of the target is applied when joining associations" do
114
- author = Author.create! name: "Jon"
115
- author.categorizations.create!
116
- author.categorizations.create! special: true
117
-
118
- assert_equal [author], Author.where(id: author).joins(:special_categorizations)
119
- end
120
-
121
- test "the default scope of the target is correctly aliased when joining associations" do
122
- author = Author.create! name: "Jon"
123
- author.categories.create! name: 'Not Special'
124
- author.special_categories.create! name: 'Special'
125
-
126
- categories = author.categories.includes(:special_categorizations).references(:special_categorizations).to_a
127
- assert_equal 2, categories.size
128
- end
129
-
130
- test "the correct records are loaded when including an aliased association" do
131
- author = Author.create! name: "Jon"
132
- author.categories.create! name: 'Not Special'
133
- author.special_categories.create! name: 'Special'
134
-
135
- categories = author.categories.eager_load(:special_categorizations).order(:name).to_a
136
- assert_equal 0, categories.first.special_categorizations.size
137
- assert_equal 1, categories.second.special_categorizations.size
138
- end
139
- end
1
+ require "cases/helper"
2
+ require 'models/post'
3
+ require 'models/comment'
4
+ require 'models/author'
5
+ require 'models/essay'
6
+ require 'models/category'
7
+ require 'models/categorization'
8
+ require 'models/person'
9
+ require 'models/tagging'
10
+ require 'models/tag'
11
+
12
+ class InnerJoinAssociationTest < ActiveRecord::TestCase
13
+ fixtures :authors, :essays, :posts, :comments, :categories, :categories_posts, :categorizations,
14
+ :taggings, :tags
15
+
16
+ def test_construct_finder_sql_applies_aliases_tables_on_association_conditions
17
+ result = Author.joins(:thinking_posts, :welcome_posts).to_a
18
+ assert_equal authors(:david), result.first
19
+ end
20
+
21
+ def test_construct_finder_sql_does_not_table_name_collide_on_duplicate_associations
22
+ assert_nothing_raised do
23
+ sql = Person.joins(:agents => {:agents => :agents}).joins(:agents => {:agents => {:primary_contact => :agents}}).to_sql
24
+ assert_match(/agents_people_4/i, sql)
25
+ end
26
+ end
27
+
28
+ def test_construct_finder_sql_ignores_empty_joins_hash
29
+ sql = Author.joins({}).to_sql
30
+ assert_no_match(/JOIN/i, sql)
31
+ end
32
+
33
+ def test_construct_finder_sql_ignores_empty_joins_array
34
+ sql = Author.joins([]).to_sql
35
+ assert_no_match(/JOIN/i, sql)
36
+ end
37
+
38
+ def test_join_conditions_added_to_join_clause
39
+ sql = Author.joins(:essays).to_sql
40
+ assert_match(/writer_type.*?=.*?Author/i, sql)
41
+ assert_no_match(/WHERE/i, sql)
42
+ end
43
+
44
+ def test_join_association_conditions_support_string_and_arel_expressions
45
+ assert_equal 0, Author.joins(:welcome_posts_with_one_comment).count
46
+ assert_equal 1, Author.joins(:welcome_posts_with_comments).count
47
+ end
48
+
49
+ def test_join_conditions_allow_nil_associations
50
+ authors = Author.includes(:essays).where(:essays => {:id => nil})
51
+ assert_equal 2, authors.count
52
+ end
53
+
54
+ def test_find_with_implicit_inner_joins_without_select_does_not_imply_readonly
55
+ authors = Author.joins(:posts)
56
+ assert_not authors.empty?, "expected authors to be non-empty"
57
+ assert authors.none?(&:readonly?), "expected no authors to be readonly"
58
+ end
59
+
60
+ def test_find_with_implicit_inner_joins_honors_readonly_with_select
61
+ authors = Author.joins(:posts).select('authors.*').to_a
62
+ assert !authors.empty?, "expected authors to be non-empty"
63
+ assert authors.all? {|a| !a.readonly? }, "expected no authors to be readonly"
64
+ end
65
+
66
+ def test_find_with_implicit_inner_joins_honors_readonly_false
67
+ authors = Author.joins(:posts).readonly(false).to_a
68
+ assert !authors.empty?, "expected authors to be non-empty"
69
+ assert authors.all? {|a| !a.readonly? }, "expected no authors to be readonly"
70
+ end
71
+
72
+ def test_find_with_implicit_inner_joins_does_not_set_associations
73
+ authors = Author.joins(:posts).select('authors.*').to_a
74
+ assert !authors.empty?, "expected authors to be non-empty"
75
+ assert authors.all? { |a| !a.instance_variable_defined?(:@posts) }, "expected no authors to have the @posts association loaded"
76
+ end
77
+
78
+ def test_count_honors_implicit_inner_joins
79
+ real_count = Author.all.to_a.sum{|a| a.posts.count }
80
+ assert_equal real_count, Author.joins(:posts).count, "plain inner join count should match the number of referenced posts records"
81
+ end
82
+
83
+ def test_calculate_honors_implicit_inner_joins
84
+ real_count = Author.all.to_a.sum{|a| a.posts.count }
85
+ assert_equal real_count, Author.joins(:posts).calculate(:count, 'authors.id'), "plain inner join count should match the number of referenced posts records"
86
+ end
87
+
88
+ def test_calculate_honors_implicit_inner_joins_and_distinct_and_conditions
89
+ real_count = Author.all.to_a.select {|a| a.posts.any? {|p| p.title =~ /^Welcome/} }.length
90
+ authors_with_welcoming_post_titles = Author.all.merge!(joins: :posts, where: "posts.title like 'Welcome%'").distinct.calculate(:count, 'authors.id')
91
+ assert_equal real_count, authors_with_welcoming_post_titles, "inner join and conditions should have only returned authors posting titles starting with 'Welcome'"
92
+ end
93
+
94
+ def test_find_with_sti_join
95
+ scope = Post.joins(:special_comments).where(:id => posts(:sti_comments).id)
96
+
97
+ # The join should match SpecialComment and its subclasses only
98
+ assert scope.where("comments.type" => "Comment").empty?
99
+ assert !scope.where("comments.type" => "SpecialComment").empty?
100
+ assert !scope.where("comments.type" => "SubSpecialComment").empty?
101
+ end
102
+
103
+ def test_find_with_conditions_on_reflection
104
+ assert !posts(:welcome).comments.empty?
105
+ assert Post.joins(:nonexistent_comments).where(:id => posts(:welcome).id).empty? # [sic!]
106
+ end
107
+
108
+ def test_find_with_conditions_on_through_reflection
109
+ assert !posts(:welcome).tags.empty?
110
+ assert Post.joins(:misc_tags).where(:id => posts(:welcome).id).empty?
111
+ end
112
+
113
+ test "the default scope of the target is applied when joining associations" do
114
+ author = Author.create! name: "Jon"
115
+ author.categorizations.create!
116
+ author.categorizations.create! special: true
117
+
118
+ assert_equal [author], Author.where(id: author).joins(:special_categorizations)
119
+ end
120
+
121
+ test "the default scope of the target is correctly aliased when joining associations" do
122
+ author = Author.create! name: "Jon"
123
+ author.categories.create! name: 'Not Special'
124
+ author.special_categories.create! name: 'Special'
125
+
126
+ categories = author.categories.includes(:special_categorizations).references(:special_categorizations).to_a
127
+ assert_equal 2, categories.size
128
+ end
129
+
130
+ test "the correct records are loaded when including an aliased association" do
131
+ author = Author.create! name: "Jon"
132
+ author.categories.create! name: 'Not Special'
133
+ author.special_categories.create! name: 'Special'
134
+
135
+ categories = author.categories.eager_load(:special_categorizations).order(:name).to_a
136
+ assert_equal 0, categories.first.special_categorizations.size
137
+ assert_equal 1, categories.second.special_categorizations.size
138
+ end
139
+ end
@@ -1,706 +1,733 @@
1
- require "cases/helper"
2
- require 'models/man'
3
- require 'models/face'
4
- require 'models/interest'
5
- require 'models/zine'
6
- require 'models/club'
7
- require 'models/sponsor'
8
- require 'models/rating'
9
- require 'models/comment'
10
- require 'models/car'
11
- require 'models/bulb'
12
- require 'models/mixed_case_monkey'
13
- require 'models/admin'
14
- require 'models/admin/account'
15
- require 'models/admin/user'
16
- require 'models/developer'
17
- require 'models/company'
18
- require 'models/project'
19
-
20
- class AutomaticInverseFindingTests < ActiveRecord::TestCase
21
- fixtures :ratings, :comments, :cars
22
-
23
- def test_has_one_and_belongs_to_should_find_inverse_automatically_on_multiple_word_name
24
- monkey_reflection = MixedCaseMonkey.reflect_on_association(:man)
25
- man_reflection = Man.reflect_on_association(:mixed_case_monkey)
26
-
27
- assert_respond_to monkey_reflection, :has_inverse?
28
- assert monkey_reflection.has_inverse?, "The monkey reflection should have an inverse"
29
- assert_equal man_reflection, monkey_reflection.inverse_of, "The monkey reflection's inverse should be the man reflection"
30
-
31
- assert_respond_to man_reflection, :has_inverse?
32
- assert man_reflection.has_inverse?, "The man reflection should have an inverse"
33
- assert_equal monkey_reflection, man_reflection.inverse_of, "The man reflection's inverse should be the monkey reflection"
34
- end
35
-
36
- def test_has_many_and_belongs_to_should_find_inverse_automatically_for_model_in_module
37
- account_reflection = Admin::Account.reflect_on_association(:users)
38
- user_reflection = Admin::User.reflect_on_association(:account)
39
-
40
- assert_respond_to account_reflection, :has_inverse?
41
- assert account_reflection.has_inverse?, "The Admin::Account reflection should have an inverse"
42
- assert_equal user_reflection, account_reflection.inverse_of, "The Admin::Account reflection's inverse should be the Admin::User reflection"
43
- end
44
-
45
- def test_has_one_and_belongs_to_should_find_inverse_automatically
46
- car_reflection = Car.reflect_on_association(:bulb)
47
- bulb_reflection = Bulb.reflect_on_association(:car)
48
-
49
- assert_respond_to car_reflection, :has_inverse?
50
- assert car_reflection.has_inverse?, "The Car reflection should have an inverse"
51
- assert_equal bulb_reflection, car_reflection.inverse_of, "The Car reflection's inverse should be the Bulb reflection"
52
-
53
- assert_respond_to bulb_reflection, :has_inverse?
54
- assert bulb_reflection.has_inverse?, "The Bulb reflection should have an inverse"
55
- assert_equal car_reflection, bulb_reflection.inverse_of, "The Bulb reflection's inverse should be the Car reflection"
56
- end
57
-
58
- def test_has_many_and_belongs_to_should_find_inverse_automatically
59
- comment_reflection = Comment.reflect_on_association(:ratings)
60
- rating_reflection = Rating.reflect_on_association(:comment)
61
-
62
- assert_respond_to comment_reflection, :has_inverse?
63
- assert comment_reflection.has_inverse?, "The Comment reflection should have an inverse"
64
- assert_equal rating_reflection, comment_reflection.inverse_of, "The Comment reflection's inverse should be the Rating reflection"
65
- end
66
-
67
- def test_has_one_and_belongs_to_automatic_inverse_shares_objects
68
- car = Car.first
69
- bulb = Bulb.create!(car: car)
70
-
71
- assert_equal car.bulb, bulb, "The Car's bulb should be the original bulb"
72
-
73
- car.bulb.color = "Blue"
74
- assert_equal car.bulb.color, bulb.color, "Changing the bulb's color on the car association should change the bulb's color"
75
-
76
- bulb.color = "Red"
77
- assert_equal bulb.color, car.bulb.color, "Changing the bulb's color should change the bulb's color on the car association"
78
- end
79
-
80
- def test_has_many_and_belongs_to_automatic_inverse_shares_objects_on_rating
81
- comment = Comment.first
82
- rating = Rating.create!(comment: comment)
83
-
84
- assert_equal rating.comment, comment, "The Rating's comment should be the original Comment"
85
-
86
- rating.comment.body = "Brogramming is the act of programming, like a bro."
87
- assert_equal rating.comment.body, comment.body, "Changing the Comment's body on the association should change the original Comment's body"
88
-
89
- comment.body = "Broseiden is the king of the sea of bros."
90
- assert_equal comment.body, rating.comment.body, "Changing the original Comment's body should change the Comment's body on the association"
91
- end
92
-
93
- def test_has_many_and_belongs_to_automatic_inverse_shares_objects_on_comment
94
- rating = Rating.create!
95
- comment = Comment.first
96
- rating.comment = comment
97
-
98
- assert_equal rating.comment, comment, "The Rating's comment should be the original Comment"
99
-
100
- rating.comment.body = "Brogramming is the act of programming, like a bro."
101
- assert_equal rating.comment.body, comment.body, "Changing the Comment's body on the association should change the original Comment's body"
102
-
103
- comment.body = "Broseiden is the king of the sea of bros."
104
- assert_equal comment.body, rating.comment.body, "Changing the original Comment's body should change the Comment's body on the association"
105
- end
106
-
107
- def test_polymorphic_and_has_many_through_relationships_should_not_have_inverses
108
- sponsor_reflection = Sponsor.reflect_on_association(:sponsorable)
109
-
110
- assert_respond_to sponsor_reflection, :has_inverse?
111
- assert !sponsor_reflection.has_inverse?, "A polymorphic association should not find an inverse automatically"
112
-
113
- club_reflection = Club.reflect_on_association(:members)
114
-
115
- assert_respond_to club_reflection, :has_inverse?
116
- assert !club_reflection.has_inverse?, "A has_many_through association should not find an inverse automatically"
117
- end
118
-
119
- def test_polymorphic_relationships_should_still_not_have_inverses_when_non_polymorphic_relationship_has_the_same_name
120
- man_reflection = Man.reflect_on_association(:polymorphic_face_without_inverse)
121
- face_reflection = Face.reflect_on_association(:man)
122
-
123
- assert_respond_to face_reflection, :has_inverse?
124
- assert face_reflection.has_inverse?, "For this test, the non-polymorphic association must have an inverse"
125
-
126
- assert_respond_to man_reflection, :has_inverse?
127
- assert !man_reflection.has_inverse?, "The target of a polymorphic association should not find an inverse automatically"
128
- end
129
- end
130
-
131
- class InverseAssociationTests < ActiveRecord::TestCase
132
- def test_should_allow_for_inverse_of_options_in_associations
133
- assert_nothing_raised(ArgumentError, 'ActiveRecord should allow the inverse_of options on has_many') do
134
- Class.new(ActiveRecord::Base).has_many(:wheels, :inverse_of => :car)
135
- end
136
-
137
- assert_nothing_raised(ArgumentError, 'ActiveRecord should allow the inverse_of options on has_one') do
138
- Class.new(ActiveRecord::Base).has_one(:engine, :inverse_of => :car)
139
- end
140
-
141
- assert_nothing_raised(ArgumentError, 'ActiveRecord should allow the inverse_of options on belongs_to') do
142
- Class.new(ActiveRecord::Base).belongs_to(:car, :inverse_of => :driver)
143
- end
144
- end
145
-
146
- def test_should_be_able_to_ask_a_reflection_if_it_has_an_inverse
147
- has_one_with_inverse_ref = Man.reflect_on_association(:face)
148
- assert_respond_to has_one_with_inverse_ref, :has_inverse?
149
- assert has_one_with_inverse_ref.has_inverse?
150
-
151
- has_many_with_inverse_ref = Man.reflect_on_association(:interests)
152
- assert_respond_to has_many_with_inverse_ref, :has_inverse?
153
- assert has_many_with_inverse_ref.has_inverse?
154
-
155
- belongs_to_with_inverse_ref = Face.reflect_on_association(:man)
156
- assert_respond_to belongs_to_with_inverse_ref, :has_inverse?
157
- assert belongs_to_with_inverse_ref.has_inverse?
158
-
159
- has_one_without_inverse_ref = Club.reflect_on_association(:sponsor)
160
- assert_respond_to has_one_without_inverse_ref, :has_inverse?
161
- assert !has_one_without_inverse_ref.has_inverse?
162
-
163
- has_many_without_inverse_ref = Club.reflect_on_association(:memberships)
164
- assert_respond_to has_many_without_inverse_ref, :has_inverse?
165
- assert !has_many_without_inverse_ref.has_inverse?
166
-
167
- belongs_to_without_inverse_ref = Sponsor.reflect_on_association(:sponsor_club)
168
- assert_respond_to belongs_to_without_inverse_ref, :has_inverse?
169
- assert !belongs_to_without_inverse_ref.has_inverse?
170
- end
171
-
172
- def test_should_be_able_to_ask_a_reflection_what_it_is_the_inverse_of
173
- has_one_ref = Man.reflect_on_association(:face)
174
- assert_respond_to has_one_ref, :inverse_of
175
-
176
- has_many_ref = Man.reflect_on_association(:interests)
177
- assert_respond_to has_many_ref, :inverse_of
178
-
179
- belongs_to_ref = Face.reflect_on_association(:man)
180
- assert_respond_to belongs_to_ref, :inverse_of
181
- end
182
-
183
- def test_inverse_of_method_should_supply_the_actual_reflection_instance_it_is_the_inverse_of
184
- has_one_ref = Man.reflect_on_association(:face)
185
- assert_equal Face.reflect_on_association(:man), has_one_ref.inverse_of
186
-
187
- has_many_ref = Man.reflect_on_association(:interests)
188
- assert_equal Interest.reflect_on_association(:man), has_many_ref.inverse_of
189
-
190
- belongs_to_ref = Face.reflect_on_association(:man)
191
- assert_equal Man.reflect_on_association(:face), belongs_to_ref.inverse_of
192
- end
193
-
194
- def test_associations_with_no_inverse_of_should_return_nil
195
- has_one_ref = Club.reflect_on_association(:sponsor)
196
- assert_nil has_one_ref.inverse_of
197
-
198
- has_many_ref = Club.reflect_on_association(:memberships)
199
- assert_nil has_many_ref.inverse_of
200
-
201
- belongs_to_ref = Sponsor.reflect_on_association(:sponsor_club)
202
- assert_nil belongs_to_ref.inverse_of
203
- end
204
-
205
- def test_this_inverse_stuff
206
- firm = Firm.create!(name: 'Adequate Holdings')
207
- Project.create!(name: 'Project 1', firm: firm)
208
- Developer.create!(name: 'Gorbypuff', firm: firm)
209
-
210
- new_project = Project.last
211
- assert Project.reflect_on_association(:lead_developer).inverse_of.present?, "Expected inverse of to be present"
212
- assert new_project.lead_developer.present?, "Expected lead developer to be present on the project"
213
- end
214
- end
215
-
216
- class InverseHasOneTests < ActiveRecord::TestCase
217
- fixtures :men, :faces
218
-
219
- def test_parent_instance_should_be_shared_with_child_on_find
220
- m = men(:gordon)
221
- f = m.face
222
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
223
- m.name = 'Bongo'
224
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
225
- f.man.name = 'Mungo'
226
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
227
- end
228
-
229
-
230
- def test_parent_instance_should_be_shared_with_eager_loaded_child_on_find
231
- m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :face).first
232
- f = m.face
233
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
234
- m.name = 'Bongo'
235
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
236
- f.man.name = 'Mungo'
237
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
238
-
239
- m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :face, :order => 'faces.id').first
240
- f = m.face
241
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
242
- m.name = 'Bongo'
243
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
244
- f.man.name = 'Mungo'
245
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
246
- end
247
-
248
- def test_parent_instance_should_be_shared_with_newly_built_child
249
- m = Man.first
250
- f = m.build_face(:description => 'haunted')
251
- assert_not_nil f.man
252
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
253
- m.name = 'Bongo'
254
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
255
- f.man.name = 'Mungo'
256
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to just-built-child-owned instance"
257
- end
258
-
259
- def test_parent_instance_should_be_shared_with_newly_created_child
260
- m = Man.first
261
- f = m.create_face(:description => 'haunted')
262
- assert_not_nil f.man
263
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
264
- m.name = 'Bongo'
265
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
266
- f.man.name = 'Mungo'
267
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
268
- end
269
-
270
- def test_parent_instance_should_be_shared_with_newly_created_child_via_bang_method
271
- m = Man.first
272
- f = m.create_face!(:description => 'haunted')
273
- assert_not_nil f.man
274
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
275
- m.name = 'Bongo'
276
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
277
- f.man.name = 'Mungo'
278
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
279
- end
280
-
281
- def test_parent_instance_should_be_shared_with_replaced_via_accessor_child
282
- m = Man.first
283
- f = Face.new(:description => 'haunted')
284
- m.face = f
285
- assert_not_nil f.man
286
- assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
287
- m.name = 'Bongo'
288
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
289
- f.man.name = 'Mungo'
290
- assert_equal m.name, f.man.name, "Name of man should be the same after changes to replaced-child-owned instance"
291
- end
292
-
293
- def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
294
- assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Man.first.dirty_face }
295
- end
296
- end
297
-
298
- class InverseHasManyTests < ActiveRecord::TestCase
299
- fixtures :men, :interests
300
-
301
- def test_parent_instance_should_be_shared_with_every_child_on_find
302
- m = men(:gordon)
303
- is = m.interests
304
- is.each do |i|
305
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
306
- m.name = 'Bongo'
307
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
308
- i.man.name = 'Mungo'
309
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
310
- end
311
- end
312
-
313
- def test_parent_instance_should_be_shared_with_eager_loaded_children
314
- m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :interests).first
315
- is = m.interests
316
- is.each do |i|
317
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
318
- m.name = 'Bongo'
319
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
320
- i.man.name = 'Mungo'
321
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
322
- end
323
-
324
- m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :interests, :order => 'interests.id').first
325
- is = m.interests
326
- is.each do |i|
327
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
328
- m.name = 'Bongo'
329
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
330
- i.man.name = 'Mungo'
331
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
332
- end
333
- end
334
-
335
- def test_parent_instance_should_be_shared_with_newly_block_style_built_child
336
- m = Man.first
337
- i = m.interests.build {|ii| ii.topic = 'Industrial Revolution Re-enactment'}
338
- assert_not_nil i.topic, "Child attributes supplied to build via blocks should be populated"
339
- assert_not_nil i.man
340
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
341
- m.name = 'Bongo'
342
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
343
- i.man.name = 'Mungo'
344
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to just-built-child-owned instance"
345
- end
346
-
347
- def test_parent_instance_should_be_shared_with_newly_created_via_bang_method_child
348
- m = Man.first
349
- i = m.interests.create!(:topic => 'Industrial Revolution Re-enactment')
350
- assert_not_nil i.man
351
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
352
- m.name = 'Bongo'
353
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
354
- i.man.name = 'Mungo'
355
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
356
- end
357
-
358
- def test_parent_instance_should_be_shared_with_newly_block_style_created_child
359
- m = Man.first
360
- i = m.interests.create {|ii| ii.topic = 'Industrial Revolution Re-enactment'}
361
- assert_not_nil i.topic, "Child attributes supplied to create via blocks should be populated"
362
- assert_not_nil i.man
363
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
364
- m.name = 'Bongo'
365
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
366
- i.man.name = 'Mungo'
367
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
368
- end
369
-
370
- def test_parent_instance_should_be_shared_within_create_block_of_new_child
371
- man = Man.first
372
- interest = man.interests.create do |i|
373
- assert i.man.equal?(man), "Man of child should be the same instance as a parent"
374
- end
375
- assert interest.man.equal?(man), "Man of the child should still be the same instance as a parent"
376
- end
377
-
378
- def test_parent_instance_should_be_shared_within_build_block_of_new_child
379
- man = Man.first
380
- interest = man.interests.build do |i|
381
- assert i.man.equal?(man), "Man of child should be the same instance as a parent"
382
- end
383
- assert interest.man.equal?(man), "Man of the child should still be the same instance as a parent"
384
- end
385
-
386
- def test_parent_instance_should_be_shared_with_poked_in_child
387
- m = men(:gordon)
388
- i = Interest.create(:topic => 'Industrial Revolution Re-enactment')
389
- m.interests << i
390
- assert_not_nil i.man
391
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
392
- m.name = 'Bongo'
393
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
394
- i.man.name = 'Mungo'
395
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
396
- end
397
-
398
- def test_parent_instance_should_be_shared_with_replaced_via_accessor_children
399
- m = Man.first
400
- i = Interest.new(:topic => 'Industrial Revolution Re-enactment')
401
- m.interests = [i]
402
- assert_not_nil i.man
403
- assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
404
- m.name = 'Bongo'
405
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
406
- i.man.name = 'Mungo'
407
- assert_equal m.name, i.man.name, "Name of man should be the same after changes to replaced-child-owned instance"
408
- end
409
-
410
- def test_parent_instance_should_be_shared_with_first_and_last_child
411
- man = Man.first
412
-
413
- assert man.interests.first.man.equal? man
414
- assert man.interests.last.man.equal? man
415
- end
416
-
417
- def test_parent_instance_should_be_shared_with_first_n_and_last_n_children
418
- man = Man.first
419
-
420
- interests = man.interests.first(2)
421
- assert interests[0].man.equal? man
422
- assert interests[1].man.equal? man
423
-
424
- interests = man.interests.last(2)
425
- assert interests[0].man.equal? man
426
- assert interests[1].man.equal? man
427
- end
428
-
429
- def test_parent_instance_should_find_child_instance_using_child_instance_id
430
- man = Man.create!
431
- interest = Interest.create!
432
- man.interests = [interest]
433
-
434
- assert interest.equal?(man.interests.first), "The inverse association should use the interest already created and held in memory"
435
- assert interest.equal?(man.interests.find(interest.id)), "The inverse association should use the interest already created and held in memory"
436
- assert man.equal?(man.interests.first.man), "Two inversion should lead back to the same object that was originally held"
437
- assert man.equal?(man.interests.find(interest.id).man), "Two inversions should lead back to the same object that was originally held"
438
- end
439
-
440
- def test_parent_instance_should_find_child_instance_using_child_instance_id_when_created
441
- man = Man.create!
442
- interest = Interest.create!(man: man)
443
-
444
- assert man.equal?(man.interests.first.man), "Two inverses should lead back to the same object that was originally held"
445
- assert man.equal?(man.interests.find(interest.id).man), "Two inversions should lead back to the same object that was originally held"
446
-
447
- assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match before the name is changed"
448
- man.name = "Ben Bitdiddle"
449
- assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match after the parent name is changed"
450
- man.interests.find(interest.id).man.name = "Alyssa P. Hacker"
451
- assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match after the child name is changed"
452
- end
453
-
454
- def test_find_on_child_instance_with_id_should_not_load_all_child_records
455
- man = Man.create!
456
- interest = Interest.create!(man: man)
457
-
458
- man.interests.find(interest.id)
459
- assert_not man.interests.loaded?
460
- end
461
-
462
- def test_raise_record_not_found_error_when_invalid_ids_are_passed
463
- # delete all interest records to ensure that hard coded invalid_id(s)
464
- # are indeed invalid.
465
- Interest.delete_all
466
-
467
- man = Man.create!
468
-
469
- invalid_id = 245324523
470
- assert_raise(ActiveRecord::RecordNotFound) { man.interests.find(invalid_id) }
471
-
472
- invalid_ids = [8432342, 2390102913, 2453245234523452]
473
- assert_raise(ActiveRecord::RecordNotFound) { man.interests.find(invalid_ids) }
474
- end
475
-
476
- def test_raise_record_not_found_error_when_no_ids_are_passed
477
- man = Man.create!
478
-
479
- assert_raise(ActiveRecord::RecordNotFound) { man.interests.find() }
480
- end
481
-
482
- def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
483
- assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Man.first.secret_interests }
484
- end
485
-
486
- def test_child_instance_should_point_to_parent_without_saving
487
- man = Man.new
488
- i = Interest.create(:topic => 'Industrial Revolution Re-enactment')
489
-
490
- man.interests << i
491
- assert_not_nil i.man
492
-
493
- i.man.name = "Charles"
494
- assert_equal i.man.name, man.name
495
-
496
- assert !man.persisted?
497
- end
498
- end
499
-
500
- class InverseBelongsToTests < ActiveRecord::TestCase
501
- fixtures :men, :faces, :interests
502
-
503
- def test_child_instance_should_be_shared_with_parent_on_find
504
- f = faces(:trusting)
505
- m = f.man
506
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
507
- f.description = 'gormless'
508
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
509
- m.face.description = 'pleasing'
510
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
511
- end
512
-
513
- def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
514
- f = Face.all.merge!(:includes => :man, :where => {:description => 'trusting'}).first
515
- m = f.man
516
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
517
- f.description = 'gormless'
518
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
519
- m.face.description = 'pleasing'
520
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
521
-
522
- f = Face.all.merge!(:includes => :man, :order => 'men.id', :where => {:description => 'trusting'}).first
523
- m = f.man
524
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
525
- f.description = 'gormless'
526
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
527
- m.face.description = 'pleasing'
528
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
529
- end
530
-
531
- def test_child_instance_should_be_shared_with_newly_built_parent
532
- f = faces(:trusting)
533
- m = f.build_man(:name => 'Charles')
534
- assert_not_nil m.face
535
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
536
- f.description = 'gormless'
537
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
538
- m.face.description = 'pleasing'
539
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to just-built-parent-owned instance"
540
- end
541
-
542
- def test_child_instance_should_be_shared_with_newly_created_parent
543
- f = faces(:trusting)
544
- m = f.create_man(:name => 'Charles')
545
- assert_not_nil m.face
546
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
547
- f.description = 'gormless'
548
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
549
- m.face.description = 'pleasing'
550
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to newly-created-parent-owned instance"
551
- end
552
-
553
- def test_should_not_try_to_set_inverse_instances_when_the_inverse_is_a_has_many
554
- i = interests(:trainspotting)
555
- m = i.man
556
- assert_not_nil m.interests
557
- iz = m.interests.detect { |_iz| _iz.id == i.id}
558
- assert_not_nil iz
559
- assert_equal i.topic, iz.topic, "Interest topics should be the same before changes to child"
560
- i.topic = 'Eating cheese with a spoon'
561
- assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to child"
562
- iz.topic = 'Cow tipping'
563
- assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to parent-owned instance"
564
- end
565
-
566
- def test_child_instance_should_be_shared_with_replaced_via_accessor_parent
567
- f = Face.first
568
- m = Man.new(:name => 'Charles')
569
- f.man = m
570
- assert_not_nil m.face
571
- assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
572
- f.description = 'gormless'
573
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
574
- m.face.description = 'pleasing'
575
- assert_equal f.description, m.face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
576
- end
577
-
578
- def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
579
- assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.horrible_man }
580
- end
581
- end
582
-
583
- class InversePolymorphicBelongsToTests < ActiveRecord::TestCase
584
- fixtures :men, :faces, :interests
585
-
586
- def test_child_instance_should_be_shared_with_parent_on_find
587
- f = Face.all.merge!(:where => {:description => 'confused'}).first
588
- m = f.polymorphic_man
589
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
590
- f.description = 'gormless'
591
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
592
- m.polymorphic_face.description = 'pleasing'
593
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
594
- end
595
-
596
- def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
597
- f = Face.all.merge!(:where => {:description => 'confused'}, :includes => :man).first
598
- m = f.polymorphic_man
599
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
600
- f.description = 'gormless'
601
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
602
- m.polymorphic_face.description = 'pleasing'
603
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
604
-
605
- f = Face.all.merge!(:where => {:description => 'confused'}, :includes => :man, :order => 'men.id').first
606
- m = f.polymorphic_man
607
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
608
- f.description = 'gormless'
609
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
610
- m.polymorphic_face.description = 'pleasing'
611
- assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
612
- end
613
-
614
- def test_child_instance_should_be_shared_with_replaced_via_accessor_parent
615
- face = faces(:confused)
616
- new_man = Man.new
617
-
618
- assert_not_nil face.polymorphic_man
619
- face.polymorphic_man = new_man
620
-
621
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same before changes to parent instance"
622
- face.description = 'Bongo'
623
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to parent instance"
624
- new_man.polymorphic_face.description = 'Mungo'
625
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
626
- end
627
-
628
- def test_child_instance_should_be_shared_with_replaced_via_method_parent
629
- face = faces(:confused)
630
- new_man = Man.new
631
-
632
- assert_not_nil face.polymorphic_man
633
- face.polymorphic_man = new_man
634
-
635
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same before changes to parent instance"
636
- face.description = 'Bongo'
637
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to parent instance"
638
- new_man.polymorphic_face.description = 'Mungo'
639
- assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
640
- end
641
-
642
- def test_inversed_instance_should_not_be_reloaded_after_stale_state_changed
643
- new_man = Man.new
644
- face = Face.new
645
- new_man.face = face
646
-
647
- old_inversed_man = face.man
648
- new_man.save!
649
- new_inversed_man = face.man
650
-
651
- assert_equal old_inversed_man.object_id, new_inversed_man.object_id
652
- end
653
-
654
- def test_should_not_try_to_set_inverse_instances_when_the_inverse_is_a_has_many
655
- i = interests(:llama_wrangling)
656
- m = i.polymorphic_man
657
- assert_not_nil m.polymorphic_interests
658
- iz = m.polymorphic_interests.detect { |_iz| _iz.id == i.id}
659
- assert_not_nil iz
660
- assert_equal i.topic, iz.topic, "Interest topics should be the same before changes to child"
661
- i.topic = 'Eating cheese with a spoon'
662
- assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to child"
663
- iz.topic = 'Cow tipping'
664
- assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to parent-owned instance"
665
- end
666
-
667
- def test_trying_to_access_inverses_that_dont_exist_shouldnt_raise_an_error
668
- # Ideally this would, if only for symmetry's sake with other association types
669
- assert_nothing_raised(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.horrible_polymorphic_man }
670
- end
671
-
672
- def test_trying_to_set_polymorphic_inverses_that_dont_exist_at_all_should_raise_an_error
673
- # fails because no class has the correct inverse_of for horrible_polymorphic_man
674
- assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.horrible_polymorphic_man = Man.first }
675
- end
676
-
677
- def test_trying_to_set_polymorphic_inverses_that_dont_exist_on_the_instance_being_set_should_raise_an_error
678
- # passes because Man does have the correct inverse_of
679
- assert_nothing_raised(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.polymorphic_man = Man.first }
680
- # fails because Interest does have the correct inverse_of
681
- assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.polymorphic_man = Interest.first }
682
- end
683
- end
684
-
685
- # NOTE - these tests might not be meaningful, ripped as they were from the parental_control plugin
686
- # which would guess the inverse rather than look for an explicit configuration option.
687
- class InverseMultipleHasManyInversesForSameModel < ActiveRecord::TestCase
688
- fixtures :men, :interests, :zines
689
-
690
- def test_that_we_can_load_associations_that_have_the_same_reciprocal_name_from_different_models
691
- assert_nothing_raised(ActiveRecord::AssociationTypeMismatch) do
692
- i = Interest.first
693
- i.zine
694
- i.man
695
- end
696
- end
697
-
698
- def test_that_we_can_create_associations_that_have_the_same_reciprocal_name_from_different_models
699
- assert_nothing_raised(ActiveRecord::AssociationTypeMismatch) do
700
- i = Interest.first
701
- i.build_zine(:title => 'Get Some in Winter! 2008')
702
- i.build_man(:name => 'Gordon')
703
- i.save!
704
- end
705
- end
706
- end
1
+ require "cases/helper"
2
+ require 'models/man'
3
+ require 'models/face'
4
+ require 'models/interest'
5
+ require 'models/zine'
6
+ require 'models/club'
7
+ require 'models/sponsor'
8
+ require 'models/rating'
9
+ require 'models/comment'
10
+ require 'models/car'
11
+ require 'models/bulb'
12
+ require 'models/mixed_case_monkey'
13
+ require 'models/admin'
14
+ require 'models/admin/account'
15
+ require 'models/admin/user'
16
+ require 'models/developer'
17
+ require 'models/company'
18
+ require 'models/project'
19
+
20
+ class AutomaticInverseFindingTests < ActiveRecord::TestCase
21
+ fixtures :ratings, :comments, :cars
22
+
23
+ def test_has_one_and_belongs_to_should_find_inverse_automatically_on_multiple_word_name
24
+ monkey_reflection = MixedCaseMonkey.reflect_on_association(:man)
25
+ man_reflection = Man.reflect_on_association(:mixed_case_monkey)
26
+
27
+ assert_respond_to monkey_reflection, :has_inverse?
28
+ assert monkey_reflection.has_inverse?, "The monkey reflection should have an inverse"
29
+ assert_equal man_reflection, monkey_reflection.inverse_of, "The monkey reflection's inverse should be the man reflection"
30
+
31
+ assert_respond_to man_reflection, :has_inverse?
32
+ assert man_reflection.has_inverse?, "The man reflection should have an inverse"
33
+ assert_equal monkey_reflection, man_reflection.inverse_of, "The man reflection's inverse should be the monkey reflection"
34
+ end
35
+
36
+ def test_has_many_and_belongs_to_should_find_inverse_automatically_for_model_in_module
37
+ account_reflection = Admin::Account.reflect_on_association(:users)
38
+ user_reflection = Admin::User.reflect_on_association(:account)
39
+
40
+ assert_respond_to account_reflection, :has_inverse?
41
+ assert account_reflection.has_inverse?, "The Admin::Account reflection should have an inverse"
42
+ assert_equal user_reflection, account_reflection.inverse_of, "The Admin::Account reflection's inverse should be the Admin::User reflection"
43
+ end
44
+
45
+ def test_has_one_and_belongs_to_should_find_inverse_automatically
46
+ car_reflection = Car.reflect_on_association(:bulb)
47
+ bulb_reflection = Bulb.reflect_on_association(:car)
48
+
49
+ assert_respond_to car_reflection, :has_inverse?
50
+ assert car_reflection.has_inverse?, "The Car reflection should have an inverse"
51
+ assert_equal bulb_reflection, car_reflection.inverse_of, "The Car reflection's inverse should be the Bulb reflection"
52
+
53
+ assert_respond_to bulb_reflection, :has_inverse?
54
+ assert bulb_reflection.has_inverse?, "The Bulb reflection should have an inverse"
55
+ assert_equal car_reflection, bulb_reflection.inverse_of, "The Bulb reflection's inverse should be the Car reflection"
56
+ end
57
+
58
+ def test_has_many_and_belongs_to_should_find_inverse_automatically
59
+ comment_reflection = Comment.reflect_on_association(:ratings)
60
+ rating_reflection = Rating.reflect_on_association(:comment)
61
+
62
+ assert_respond_to comment_reflection, :has_inverse?
63
+ assert comment_reflection.has_inverse?, "The Comment reflection should have an inverse"
64
+ assert_equal rating_reflection, comment_reflection.inverse_of, "The Comment reflection's inverse should be the Rating reflection"
65
+ end
66
+
67
+ def test_has_one_and_belongs_to_automatic_inverse_shares_objects
68
+ car = Car.first
69
+ bulb = Bulb.create!(car: car)
70
+
71
+ assert_equal car.bulb, bulb, "The Car's bulb should be the original bulb"
72
+
73
+ car.bulb.color = "Blue"
74
+ assert_equal car.bulb.color, bulb.color, "Changing the bulb's color on the car association should change the bulb's color"
75
+
76
+ bulb.color = "Red"
77
+ assert_equal bulb.color, car.bulb.color, "Changing the bulb's color should change the bulb's color on the car association"
78
+ end
79
+
80
+ def test_has_many_and_belongs_to_automatic_inverse_shares_objects_on_rating
81
+ comment = Comment.first
82
+ rating = Rating.create!(comment: comment)
83
+
84
+ assert_equal rating.comment, comment, "The Rating's comment should be the original Comment"
85
+
86
+ rating.comment.body = "Fennec foxes are the smallest of the foxes."
87
+ assert_equal rating.comment.body, comment.body, "Changing the Comment's body on the association should change the original Comment's body"
88
+
89
+ comment.body = "Kittens are adorable."
90
+ assert_equal comment.body, rating.comment.body, "Changing the original Comment's body should change the Comment's body on the association"
91
+ end
92
+
93
+ def test_has_many_and_belongs_to_automatic_inverse_shares_objects_on_comment
94
+ rating = Rating.create!
95
+ comment = Comment.first
96
+ rating.comment = comment
97
+
98
+ assert_equal rating.comment, comment, "The Rating's comment should be the original Comment"
99
+
100
+ rating.comment.body = "Fennec foxes are the smallest of the foxes."
101
+ assert_equal rating.comment.body, comment.body, "Changing the Comment's body on the association should change the original Comment's body"
102
+
103
+ comment.body = "Kittens are adorable."
104
+ assert_equal comment.body, rating.comment.body, "Changing the original Comment's body should change the Comment's body on the association"
105
+ end
106
+
107
+ def test_polymorphic_and_has_many_through_relationships_should_not_have_inverses
108
+ sponsor_reflection = Sponsor.reflect_on_association(:sponsorable)
109
+
110
+ assert_respond_to sponsor_reflection, :has_inverse?
111
+ assert !sponsor_reflection.has_inverse?, "A polymorphic association should not find an inverse automatically"
112
+
113
+ club_reflection = Club.reflect_on_association(:members)
114
+
115
+ assert_respond_to club_reflection, :has_inverse?
116
+ assert !club_reflection.has_inverse?, "A has_many_through association should not find an inverse automatically"
117
+ end
118
+
119
+ def test_polymorphic_relationships_should_still_not_have_inverses_when_non_polymorphic_relationship_has_the_same_name
120
+ man_reflection = Man.reflect_on_association(:polymorphic_face_without_inverse)
121
+ face_reflection = Face.reflect_on_association(:man)
122
+
123
+ assert_respond_to face_reflection, :has_inverse?
124
+ assert face_reflection.has_inverse?, "For this test, the non-polymorphic association must have an inverse"
125
+
126
+ assert_respond_to man_reflection, :has_inverse?
127
+ assert !man_reflection.has_inverse?, "The target of a polymorphic association should not find an inverse automatically"
128
+ end
129
+ end
130
+
131
+ class InverseAssociationTests < ActiveRecord::TestCase
132
+ def test_should_allow_for_inverse_of_options_in_associations
133
+ assert_nothing_raised do
134
+ Class.new(ActiveRecord::Base).has_many(:wheels, :inverse_of => :car)
135
+ end
136
+
137
+ assert_nothing_raised do
138
+ Class.new(ActiveRecord::Base).has_one(:engine, :inverse_of => :car)
139
+ end
140
+
141
+ assert_nothing_raised do
142
+ Class.new(ActiveRecord::Base).belongs_to(:car, :inverse_of => :driver)
143
+ end
144
+ end
145
+
146
+ def test_should_be_able_to_ask_a_reflection_if_it_has_an_inverse
147
+ has_one_with_inverse_ref = Man.reflect_on_association(:face)
148
+ assert_respond_to has_one_with_inverse_ref, :has_inverse?
149
+ assert has_one_with_inverse_ref.has_inverse?
150
+
151
+ has_many_with_inverse_ref = Man.reflect_on_association(:interests)
152
+ assert_respond_to has_many_with_inverse_ref, :has_inverse?
153
+ assert has_many_with_inverse_ref.has_inverse?
154
+
155
+ belongs_to_with_inverse_ref = Face.reflect_on_association(:man)
156
+ assert_respond_to belongs_to_with_inverse_ref, :has_inverse?
157
+ assert belongs_to_with_inverse_ref.has_inverse?
158
+
159
+ has_one_without_inverse_ref = Club.reflect_on_association(:sponsor)
160
+ assert_respond_to has_one_without_inverse_ref, :has_inverse?
161
+ assert !has_one_without_inverse_ref.has_inverse?
162
+
163
+ has_many_without_inverse_ref = Club.reflect_on_association(:memberships)
164
+ assert_respond_to has_many_without_inverse_ref, :has_inverse?
165
+ assert !has_many_without_inverse_ref.has_inverse?
166
+
167
+ belongs_to_without_inverse_ref = Sponsor.reflect_on_association(:sponsor_club)
168
+ assert_respond_to belongs_to_without_inverse_ref, :has_inverse?
169
+ assert !belongs_to_without_inverse_ref.has_inverse?
170
+ end
171
+
172
+ def test_should_be_able_to_ask_a_reflection_what_it_is_the_inverse_of
173
+ has_one_ref = Man.reflect_on_association(:face)
174
+ assert_respond_to has_one_ref, :inverse_of
175
+
176
+ has_many_ref = Man.reflect_on_association(:interests)
177
+ assert_respond_to has_many_ref, :inverse_of
178
+
179
+ belongs_to_ref = Face.reflect_on_association(:man)
180
+ assert_respond_to belongs_to_ref, :inverse_of
181
+ end
182
+
183
+ def test_inverse_of_method_should_supply_the_actual_reflection_instance_it_is_the_inverse_of
184
+ has_one_ref = Man.reflect_on_association(:face)
185
+ assert_equal Face.reflect_on_association(:man), has_one_ref.inverse_of
186
+
187
+ has_many_ref = Man.reflect_on_association(:interests)
188
+ assert_equal Interest.reflect_on_association(:man), has_many_ref.inverse_of
189
+
190
+ belongs_to_ref = Face.reflect_on_association(:man)
191
+ assert_equal Man.reflect_on_association(:face), belongs_to_ref.inverse_of
192
+ end
193
+
194
+ def test_associations_with_no_inverse_of_should_return_nil
195
+ has_one_ref = Club.reflect_on_association(:sponsor)
196
+ assert_nil has_one_ref.inverse_of
197
+
198
+ has_many_ref = Club.reflect_on_association(:memberships)
199
+ assert_nil has_many_ref.inverse_of
200
+
201
+ belongs_to_ref = Sponsor.reflect_on_association(:sponsor_club)
202
+ assert_nil belongs_to_ref.inverse_of
203
+ end
204
+
205
+ def test_this_inverse_stuff
206
+ firm = Firm.create!(name: 'Adequate Holdings')
207
+ Project.create!(name: 'Project 1', firm: firm)
208
+ Developer.create!(name: 'Gorbypuff', firm: firm)
209
+
210
+ new_project = Project.last
211
+ assert Project.reflect_on_association(:lead_developer).inverse_of.present?, "Expected inverse of to be present"
212
+ assert new_project.lead_developer.present?, "Expected lead developer to be present on the project"
213
+ end
214
+ end
215
+
216
+ class InverseHasOneTests < ActiveRecord::TestCase
217
+ fixtures :men, :faces
218
+
219
+ def test_parent_instance_should_be_shared_with_child_on_find
220
+ m = men(:gordon)
221
+ f = m.face
222
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
223
+ m.name = 'Bongo'
224
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
225
+ f.man.name = 'Mungo'
226
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
227
+ end
228
+
229
+
230
+ def test_parent_instance_should_be_shared_with_eager_loaded_child_on_find
231
+ m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :face).first
232
+ f = m.face
233
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
234
+ m.name = 'Bongo'
235
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
236
+ f.man.name = 'Mungo'
237
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
238
+
239
+ m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :face, :order => 'faces.id').first
240
+ f = m.face
241
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
242
+ m.name = 'Bongo'
243
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
244
+ f.man.name = 'Mungo'
245
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
246
+ end
247
+
248
+ def test_parent_instance_should_be_shared_with_newly_built_child
249
+ m = Man.first
250
+ f = m.build_face(:description => 'haunted')
251
+ assert_not_nil f.man
252
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
253
+ m.name = 'Bongo'
254
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
255
+ f.man.name = 'Mungo'
256
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to just-built-child-owned instance"
257
+ end
258
+
259
+ def test_parent_instance_should_be_shared_with_newly_created_child
260
+ m = Man.first
261
+ f = m.create_face(:description => 'haunted')
262
+ assert_not_nil f.man
263
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
264
+ m.name = 'Bongo'
265
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
266
+ f.man.name = 'Mungo'
267
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
268
+ end
269
+
270
+ def test_parent_instance_should_be_shared_with_newly_created_child_via_bang_method
271
+ m = Man.first
272
+ f = m.create_face!(:description => 'haunted')
273
+ assert_not_nil f.man
274
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
275
+ m.name = 'Bongo'
276
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
277
+ f.man.name = 'Mungo'
278
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
279
+ end
280
+
281
+ def test_parent_instance_should_be_shared_with_replaced_via_accessor_child
282
+ m = Man.first
283
+ f = Face.new(:description => 'haunted')
284
+ m.face = f
285
+ assert_not_nil f.man
286
+ assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
287
+ m.name = 'Bongo'
288
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to parent instance"
289
+ f.man.name = 'Mungo'
290
+ assert_equal m.name, f.man.name, "Name of man should be the same after changes to replaced-child-owned instance"
291
+ end
292
+
293
+ def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
294
+ assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Man.first.dirty_face }
295
+ end
296
+ end
297
+
298
+ class InverseHasManyTests < ActiveRecord::TestCase
299
+ fixtures :men, :interests
300
+
301
+ def test_parent_instance_should_be_shared_with_every_child_on_find
302
+ m = men(:gordon)
303
+ is = m.interests
304
+ is.each do |i|
305
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
306
+ m.name = 'Bongo'
307
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
308
+ i.man.name = 'Mungo'
309
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
310
+ end
311
+ end
312
+
313
+ def test_parent_instance_should_be_shared_with_eager_loaded_children
314
+ m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :interests).first
315
+ is = m.interests
316
+ is.each do |i|
317
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
318
+ m.name = 'Bongo'
319
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
320
+ i.man.name = 'Mungo'
321
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
322
+ end
323
+
324
+ m = Man.all.merge!(:where => {:name => 'Gordon'}, :includes => :interests, :order => 'interests.id').first
325
+ is = m.interests
326
+ is.each do |i|
327
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
328
+ m.name = 'Bongo'
329
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
330
+ i.man.name = 'Mungo'
331
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
332
+ end
333
+ end
334
+
335
+ def test_parent_instance_should_be_shared_with_newly_block_style_built_child
336
+ m = Man.first
337
+ i = m.interests.build {|ii| ii.topic = 'Industrial Revolution Re-enactment'}
338
+ assert_not_nil i.topic, "Child attributes supplied to build via blocks should be populated"
339
+ assert_not_nil i.man
340
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
341
+ m.name = 'Bongo'
342
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
343
+ i.man.name = 'Mungo'
344
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to just-built-child-owned instance"
345
+ end
346
+
347
+ def test_parent_instance_should_be_shared_with_newly_created_via_bang_method_child
348
+ m = Man.first
349
+ i = m.interests.create!(:topic => 'Industrial Revolution Re-enactment')
350
+ assert_not_nil i.man
351
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
352
+ m.name = 'Bongo'
353
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
354
+ i.man.name = 'Mungo'
355
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
356
+ end
357
+
358
+ def test_parent_instance_should_be_shared_with_newly_block_style_created_child
359
+ m = Man.first
360
+ i = m.interests.create {|ii| ii.topic = 'Industrial Revolution Re-enactment'}
361
+ assert_not_nil i.topic, "Child attributes supplied to create via blocks should be populated"
362
+ assert_not_nil i.man
363
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
364
+ m.name = 'Bongo'
365
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
366
+ i.man.name = 'Mungo'
367
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
368
+ end
369
+
370
+ def test_parent_instance_should_be_shared_within_create_block_of_new_child
371
+ man = Man.first
372
+ interest = man.interests.create do |i|
373
+ assert i.man.equal?(man), "Man of child should be the same instance as a parent"
374
+ end
375
+ assert interest.man.equal?(man), "Man of the child should still be the same instance as a parent"
376
+ end
377
+
378
+ def test_parent_instance_should_be_shared_within_build_block_of_new_child
379
+ man = Man.first
380
+ interest = man.interests.build do |i|
381
+ assert i.man.equal?(man), "Man of child should be the same instance as a parent"
382
+ end
383
+ assert interest.man.equal?(man), "Man of the child should still be the same instance as a parent"
384
+ end
385
+
386
+ def test_parent_instance_should_be_shared_with_poked_in_child
387
+ m = men(:gordon)
388
+ i = Interest.create(:topic => 'Industrial Revolution Re-enactment')
389
+ m.interests << i
390
+ assert_not_nil i.man
391
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
392
+ m.name = 'Bongo'
393
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
394
+ i.man.name = 'Mungo'
395
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to newly-created-child-owned instance"
396
+ end
397
+
398
+ def test_parent_instance_should_be_shared_with_replaced_via_accessor_children
399
+ m = Man.first
400
+ i = Interest.new(:topic => 'Industrial Revolution Re-enactment')
401
+ m.interests = [i]
402
+ assert_not_nil i.man
403
+ assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
404
+ m.name = 'Bongo'
405
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to parent instance"
406
+ i.man.name = 'Mungo'
407
+ assert_equal m.name, i.man.name, "Name of man should be the same after changes to replaced-child-owned instance"
408
+ end
409
+
410
+ def test_parent_instance_should_be_shared_with_first_and_last_child
411
+ man = Man.first
412
+
413
+ assert man.interests.first.man.equal? man
414
+ assert man.interests.last.man.equal? man
415
+ end
416
+
417
+ def test_parent_instance_should_be_shared_with_first_n_and_last_n_children
418
+ man = Man.first
419
+
420
+ interests = man.interests.first(2)
421
+ assert interests[0].man.equal? man
422
+ assert interests[1].man.equal? man
423
+
424
+ interests = man.interests.last(2)
425
+ assert interests[0].man.equal? man
426
+ assert interests[1].man.equal? man
427
+ end
428
+
429
+ def test_parent_instance_should_find_child_instance_using_child_instance_id
430
+ man = Man.create!
431
+ interest = Interest.create!
432
+ man.interests = [interest]
433
+
434
+ assert interest.equal?(man.interests.first), "The inverse association should use the interest already created and held in memory"
435
+ assert interest.equal?(man.interests.find(interest.id)), "The inverse association should use the interest already created and held in memory"
436
+ assert man.equal?(man.interests.first.man), "Two inversion should lead back to the same object that was originally held"
437
+ assert man.equal?(man.interests.find(interest.id).man), "Two inversions should lead back to the same object that was originally held"
438
+ end
439
+
440
+ def test_parent_instance_should_find_child_instance_using_child_instance_id_when_created
441
+ man = Man.create!
442
+ interest = Interest.create!(man: man)
443
+
444
+ assert man.equal?(man.interests.first.man), "Two inverses should lead back to the same object that was originally held"
445
+ assert man.equal?(man.interests.find(interest.id).man), "Two inversions should lead back to the same object that was originally held"
446
+
447
+ assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match before the name is changed"
448
+ man.name = "Ben Bitdiddle"
449
+ assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match after the parent name is changed"
450
+ man.interests.find(interest.id).man.name = "Alyssa P. Hacker"
451
+ assert_equal man.name, man.interests.find(interest.id).man.name, "The name of the man should match after the child name is changed"
452
+ end
453
+
454
+ def test_find_on_child_instance_with_id_should_not_load_all_child_records
455
+ man = Man.create!
456
+ interest = Interest.create!(man: man)
457
+
458
+ man.interests.find(interest.id)
459
+ assert_not man.interests.loaded?
460
+ end
461
+
462
+ def test_raise_record_not_found_error_when_invalid_ids_are_passed
463
+ # delete all interest records to ensure that hard coded invalid_id(s)
464
+ # are indeed invalid.
465
+ Interest.delete_all
466
+
467
+ man = Man.create!
468
+
469
+ invalid_id = 245324523
470
+ assert_raise(ActiveRecord::RecordNotFound) { man.interests.find(invalid_id) }
471
+
472
+ invalid_ids = [8432342, 2390102913, 2453245234523452]
473
+ assert_raise(ActiveRecord::RecordNotFound) { man.interests.find(invalid_ids) }
474
+ end
475
+
476
+ def test_raise_record_not_found_error_when_no_ids_are_passed
477
+ man = Man.create!
478
+
479
+ assert_raise(ActiveRecord::RecordNotFound) { man.interests.find() }
480
+ end
481
+
482
+ def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
483
+ assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Man.first.secret_interests }
484
+ end
485
+
486
+ def test_child_instance_should_point_to_parent_without_saving
487
+ man = Man.new
488
+ i = Interest.create(:topic => 'Industrial Revolution Re-enactment')
489
+
490
+ man.interests << i
491
+ assert_not_nil i.man
492
+
493
+ i.man.name = "Charles"
494
+ assert_equal i.man.name, man.name
495
+
496
+ assert !man.persisted?
497
+ end
498
+
499
+ def test_inverse_instance_should_be_set_before_find_callbacks_are_run
500
+ reset_callbacks(Interest, :find) do
501
+ Interest.after_find { raise unless association(:man).loaded? && man.present? }
502
+
503
+ assert Man.first.interests.reload.any?
504
+ assert Man.includes(:interests).first.interests.any?
505
+ assert Man.joins(:interests).includes(:interests).first.interests.any?
506
+ end
507
+ end
508
+
509
+ def test_inverse_instance_should_be_set_before_initialize_callbacks_are_run
510
+ reset_callbacks(Interest, :initialize) do
511
+ Interest.after_initialize { raise unless association(:man).loaded? && man.present? }
512
+
513
+ assert Man.first.interests.reload.any?
514
+ assert Man.includes(:interests).first.interests.any?
515
+ assert Man.joins(:interests).includes(:interests).first.interests.any?
516
+ end
517
+ end
518
+
519
+ def reset_callbacks(target, type)
520
+ old_callbacks = target.send(:get_callbacks, type).deep_dup
521
+ yield
522
+ ensure
523
+ target.send(:set_callbacks, type, old_callbacks) if old_callbacks
524
+ end
525
+ end
526
+
527
+ class InverseBelongsToTests < ActiveRecord::TestCase
528
+ fixtures :men, :faces, :interests
529
+
530
+ def test_child_instance_should_be_shared_with_parent_on_find
531
+ f = faces(:trusting)
532
+ m = f.man
533
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
534
+ f.description = 'gormless'
535
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
536
+ m.face.description = 'pleasing'
537
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
538
+ end
539
+
540
+ def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
541
+ f = Face.all.merge!(:includes => :man, :where => {:description => 'trusting'}).first
542
+ m = f.man
543
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
544
+ f.description = 'gormless'
545
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
546
+ m.face.description = 'pleasing'
547
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
548
+
549
+ f = Face.all.merge!(:includes => :man, :order => 'men.id', :where => {:description => 'trusting'}).first
550
+ m = f.man
551
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
552
+ f.description = 'gormless'
553
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
554
+ m.face.description = 'pleasing'
555
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
556
+ end
557
+
558
+ def test_child_instance_should_be_shared_with_newly_built_parent
559
+ f = faces(:trusting)
560
+ m = f.build_man(:name => 'Charles')
561
+ assert_not_nil m.face
562
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
563
+ f.description = 'gormless'
564
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
565
+ m.face.description = 'pleasing'
566
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to just-built-parent-owned instance"
567
+ end
568
+
569
+ def test_child_instance_should_be_shared_with_newly_created_parent
570
+ f = faces(:trusting)
571
+ m = f.create_man(:name => 'Charles')
572
+ assert_not_nil m.face
573
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
574
+ f.description = 'gormless'
575
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
576
+ m.face.description = 'pleasing'
577
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to newly-created-parent-owned instance"
578
+ end
579
+
580
+ def test_should_not_try_to_set_inverse_instances_when_the_inverse_is_a_has_many
581
+ i = interests(:trainspotting)
582
+ m = i.man
583
+ assert_not_nil m.interests
584
+ iz = m.interests.detect { |_iz| _iz.id == i.id}
585
+ assert_not_nil iz
586
+ assert_equal i.topic, iz.topic, "Interest topics should be the same before changes to child"
587
+ i.topic = 'Eating cheese with a spoon'
588
+ assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to child"
589
+ iz.topic = 'Cow tipping'
590
+ assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to parent-owned instance"
591
+ end
592
+
593
+ def test_child_instance_should_be_shared_with_replaced_via_accessor_parent
594
+ f = Face.first
595
+ m = Man.new(:name => 'Charles')
596
+ f.man = m
597
+ assert_not_nil m.face
598
+ assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
599
+ f.description = 'gormless'
600
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to child instance"
601
+ m.face.description = 'pleasing'
602
+ assert_equal f.description, m.face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
603
+ end
604
+
605
+ def test_trying_to_use_inverses_that_dont_exist_should_raise_an_error
606
+ assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.horrible_man }
607
+ end
608
+ end
609
+
610
+ class InversePolymorphicBelongsToTests < ActiveRecord::TestCase
611
+ fixtures :men, :faces, :interests
612
+
613
+ def test_child_instance_should_be_shared_with_parent_on_find
614
+ f = Face.all.merge!(:where => {:description => 'confused'}).first
615
+ m = f.polymorphic_man
616
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
617
+ f.description = 'gormless'
618
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
619
+ m.polymorphic_face.description = 'pleasing'
620
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
621
+ end
622
+
623
+ def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
624
+ f = Face.all.merge!(:where => {:description => 'confused'}, :includes => :man).first
625
+ m = f.polymorphic_man
626
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
627
+ f.description = 'gormless'
628
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
629
+ m.polymorphic_face.description = 'pleasing'
630
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
631
+
632
+ f = Face.all.merge!(:where => {:description => 'confused'}, :includes => :man, :order => 'men.id').first
633
+ m = f.polymorphic_man
634
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
635
+ f.description = 'gormless'
636
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to child instance"
637
+ m.polymorphic_face.description = 'pleasing'
638
+ assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
639
+ end
640
+
641
+ def test_child_instance_should_be_shared_with_replaced_via_accessor_parent
642
+ face = faces(:confused)
643
+ new_man = Man.new
644
+
645
+ assert_not_nil face.polymorphic_man
646
+ face.polymorphic_man = new_man
647
+
648
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same before changes to parent instance"
649
+ face.description = 'Bongo'
650
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to parent instance"
651
+ new_man.polymorphic_face.description = 'Mungo'
652
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
653
+ end
654
+
655
+ def test_child_instance_should_be_shared_with_replaced_via_method_parent
656
+ face = faces(:confused)
657
+ new_man = Man.new
658
+
659
+ assert_not_nil face.polymorphic_man
660
+ face.polymorphic_man = new_man
661
+
662
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same before changes to parent instance"
663
+ face.description = 'Bongo'
664
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to parent instance"
665
+ new_man.polymorphic_face.description = 'Mungo'
666
+ assert_equal face.description, new_man.polymorphic_face.description, "Description of face should be the same after changes to replaced-parent-owned instance"
667
+ end
668
+
669
+ def test_inversed_instance_should_not_be_reloaded_after_stale_state_changed
670
+ new_man = Man.new
671
+ face = Face.new
672
+ new_man.face = face
673
+
674
+ old_inversed_man = face.man
675
+ new_man.save!
676
+ new_inversed_man = face.man
677
+
678
+ assert_equal old_inversed_man.object_id, new_inversed_man.object_id
679
+ end
680
+
681
+ def test_should_not_try_to_set_inverse_instances_when_the_inverse_is_a_has_many
682
+ i = interests(:llama_wrangling)
683
+ m = i.polymorphic_man
684
+ assert_not_nil m.polymorphic_interests
685
+ iz = m.polymorphic_interests.detect { |_iz| _iz.id == i.id}
686
+ assert_not_nil iz
687
+ assert_equal i.topic, iz.topic, "Interest topics should be the same before changes to child"
688
+ i.topic = 'Eating cheese with a spoon'
689
+ assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to child"
690
+ iz.topic = 'Cow tipping'
691
+ assert_not_equal i.topic, iz.topic, "Interest topics should not be the same after changes to parent-owned instance"
692
+ end
693
+
694
+ def test_trying_to_access_inverses_that_dont_exist_shouldnt_raise_an_error
695
+ # Ideally this would, if only for symmetry's sake with other association types
696
+ assert_nothing_raised { Face.first.horrible_polymorphic_man }
697
+ end
698
+
699
+ def test_trying_to_set_polymorphic_inverses_that_dont_exist_at_all_should_raise_an_error
700
+ # fails because no class has the correct inverse_of for horrible_polymorphic_man
701
+ assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.horrible_polymorphic_man = Man.first }
702
+ end
703
+
704
+ def test_trying_to_set_polymorphic_inverses_that_dont_exist_on_the_instance_being_set_should_raise_an_error
705
+ # passes because Man does have the correct inverse_of
706
+ assert_nothing_raised { Face.first.polymorphic_man = Man.first }
707
+ # fails because Interest does have the correct inverse_of
708
+ assert_raise(ActiveRecord::InverseOfAssociationNotFoundError) { Face.first.polymorphic_man = Interest.first }
709
+ end
710
+ end
711
+
712
+ # NOTE - these tests might not be meaningful, ripped as they were from the parental_control plugin
713
+ # which would guess the inverse rather than look for an explicit configuration option.
714
+ class InverseMultipleHasManyInversesForSameModel < ActiveRecord::TestCase
715
+ fixtures :men, :interests, :zines
716
+
717
+ def test_that_we_can_load_associations_that_have_the_same_reciprocal_name_from_different_models
718
+ assert_nothing_raised do
719
+ i = Interest.first
720
+ i.zine
721
+ i.man
722
+ end
723
+ end
724
+
725
+ def test_that_we_can_create_associations_that_have_the_same_reciprocal_name_from_different_models
726
+ assert_nothing_raised do
727
+ i = Interest.first
728
+ i.build_zine(:title => 'Get Some in Winter! 2008')
729
+ i.build_man(:name => 'Gordon')
730
+ i.save!
731
+ end
732
+ end
733
+ end