Re: [patch] Dependents and Dependencies in GreenPlum

Hi Sarah

On Tue, May 9, 2017 at 9:03 PM, Sarah McAlear <smcalear(at)pivotal(dot)io> wrote:

> Hi Akshay!
>
>
>> Some test file names ended with "*_sql_template.py*" do we need to add
>> that string ?
>
> we added this suffix after moving the tests up a level to tables/tests to
> clarify what subject they applied to. we changed the suffix to "_sql.py"
>
> - Files "test_column_acl_sql_template.py" and "test_column_properties_sql
>> _template.py" should be moved from tables->tests to
>> tables->column->tests. As it's related to column.
>> - Files "test_trigger_get_oid_sql_template.py" and "test_trigger_nodes_
>> sql_template.py" should be moved from tables->tests to
>> tables->triggers->tests. As its related to triggers.
>
> these tests are related to the sql files in tables/templates/column, not
> tables/column, so moving them into tables/column would be more confusing.
>
> Following test cases are failing
>
> Thank you, fixed, see new patch. Can you confirm that TestTablesNodeSql
> doesn't fail in your environment?
>

TestTablesNodeSql hasn't failed but TestRoleDependenciesSql failed with
below error:
.....\test_role_dependencies_sql.py\", line 41, in assertions\n
self.assertEqual(1, len(fetch_result))\nAssertionError: 1 != 0

> Thanks,
> George & Sarah
>
>

--
*Akshay Joshi*
*Principal Software Engineer *

*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*