{"action":"created","issue":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/815","repository_url":"https://api.github.com/repos/riptano/cassandra-dtest","labels_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/815/labels{/name}","comments_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/815/comments","events_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/815/events","html_url":"https://github.com/riptano/cassandra-dtest/pull/815","id":135456309,"number":815,"title":"Cassandra 6377","user":{"login":"blerer","id":6393833,"avatar_url":"https://avatars.githubusercontent.com/u/6393833?v=3","gravatar_id":"","url":"https://api.github.com/users/blerer","html_url":"https://github.com/blerer","followers_url":"https://api.github.com/users/blerer/followers","following_url":"https://api.github.com/users/blerer/following{/other_user}","gists_url":"https://api.github.com/users/blerer/gists{/gist_id}","starred_url":"https://api.github.com/users/blerer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/blerer/subscriptions","organizations_url":"https://api.github.com/users/blerer/orgs","repos_url":"https://api.github.com/users/blerer/repos","events_url":"https://api.github.com/users/blerer/events{/privacy}","received_events_url":"https://api.github.com/users/blerer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-02-22T15:45:17Z","updated_at":"2016-02-29T18:01:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/pulls/815","html_url":"https://github.com/riptano/cassandra-dtest/pull/815","diff_url":"https://github.com/riptano/cassandra-dtest/pull/815.diff","patch_url":"https://github.com/riptano/cassandra-dtest/pull/815.patch"},"body":"Fix the test for the CASSANDRA-6377 patch and add extra tests"},"comment":{"url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/comments/190312371","html_url":"https://github.com/riptano/cassandra-dtest/pull/815#issuecomment-190312371","issue_url":"https://api.github.com/repos/riptano/cassandra-dtest/issues/815","id":190312371,"user":{"login":"mambocab","id":3753769,"avatar_url":"https://avatars.githubusercontent.com/u/3753769?v=3","gravatar_id":"","url":"https://api.github.com/users/mambocab","html_url":"https://github.com/mambocab","followers_url":"https://api.github.com/users/mambocab/followers","following_url":"https://api.github.com/users/mambocab/following{/other_user}","gists_url":"https://api.github.com/users/mambocab/gists{/gist_id}","starred_url":"https://api.github.com/users/mambocab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mambocab/subscriptions","organizations_url":"https://api.github.com/users/mambocab/orgs","repos_url":"https://api.github.com/users/mambocab/repos","events_url":"https://api.github.com/users/mambocab/events{/privacy}","received_events_url":"https://api.github.com/users/mambocab/received_events","type":"User","site_admin":false},"created_at":"2016-02-29T18:01:52Z","updated_at":"2016-02-29T18:01:52Z","body":"I'm not sure I'm making my objection clear -- my objection is that, when a statement is executed in the dtest and fails, a bare `assert_invalid` doesn't verify that the failure happened because of a 2i-related problem, only that the underlying error generated an `InvalidRequest` exception. Unless the unit tests logically duplicate the dtests, they can't verify that failures generated in the dtests happened for the correct reason; that has to happen in the dtest itself, as far as I can tell.\r\n\r\nIn any event, does the improved assertion from this commit work?:\r\n\r\nhttps://github.com/mambocab/cassandra-dtest/commit/e038bdfa753a1537365ef311d07b8c113ea9ed87\r\n\r\nIt seems to work locally; I ran the following tests:\r\n\r\n```\r\ntests=\"nosetests secondary_indexes_test.py:TestSecondaryIndexesOnCollections.test_list_indexes secondary_indexes_test.py:TestSecondaryIndexesOnCollections.test_map_indexes secondary_indexes_test.py:TestSecondaryIndexesOnCollections.test_set_indexes secondary_indexes_test.py:TestSecondaryIndexesOnCollections.test_tuple_indexes user_types_test.py:TestUserTypes.test_type_secondary_indexing\"\r\n\r\nCASSANDRA_VERSION=github:blerer/6377-2.2 $tests --with-id --verbosity=2 --stop --nocapture\r\nCASSANDRA_VERSION=github:blerer/6377-3.0 nosetests $tests --with-id --verbosity=2 --stop --nocapture\r\nCASSANDRA_VERSION=github:blerer/6377-trunk nosetests $tests --with-id --verbosity=2 --stop --nocapture\r\n```\r\n\r\nAnd they passed, so they seem to work with the new messages introduced in CASSANDRA-6377."}}