> Looks great.
>
> We don't appear to have a glance bin test on clear all. Probably some overlap
> with existing tests but we could create a test named
> 'test_add_list_clear_list' test case or something? I approve w/ or without
> this test case however.
Sure, but keep in mind clear only deletes *public* images, since it uses a call to index() and then delete()s each one in the index (which is only public images)
> Looks great. list_clear_ list' test case or something? I approve w/ or without
>
> We don't appear to have a glance bin test on clear all. Probably some overlap
> with existing tests but we could create a test named
> 'test_add_
> this test case however.
Sure, but keep in mind clear only deletes *public* images, since it uses a call to index() and then delete()s each one in the index (which is only public images)
-jay