{"action":"created","issue":{"url":"https://api.github.com/repos/edx/edx-platform/issues/9489","labels_url":"https://api.github.com/repos/edx/edx-platform/issues/9489/labels{/name}","comments_url":"https://api.github.com/repos/edx/edx-platform/issues/9489/comments","events_url":"https://api.github.com/repos/edx/edx-platform/issues/9489/events","html_url":"https://github.com/edx/edx-platform/pull/9489","id":103348519,"number":9489,"title":"Improve teams test run times by skipping signals","user":{"login":"bderusha","id":444835,"avatar_url":"https://avatars.githubusercontent.com/u/444835?v=3","gravatar_id":"","url":"https://api.github.com/users/bderusha","html_url":"https://github.com/bderusha","followers_url":"https://api.github.com/users/bderusha/followers","following_url":"https://api.github.com/users/bderusha/following{/other_user}","gists_url":"https://api.github.com/users/bderusha/gists{/gist_id}","starred_url":"https://api.github.com/users/bderusha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bderusha/subscriptions","organizations_url":"https://api.github.com/users/bderusha/orgs","repos_url":"https://api.github.com/users/bderusha/repos","events_url":"https://api.github.com/users/bderusha/events{/privacy}","received_events_url":"https://api.github.com/users/bderusha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-26T19:39:01Z","updated_at":"2015-08-26T19:45:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/9489","html_url":"https://github.com/edx/edx-platform/pull/9489","diff_url":"https://github.com/edx/edx-platform/pull/9489.diff","patch_url":"https://github.com/edx/edx-platform/pull/9489.patch"},"body":"Ticket :ticket: https://openedx.atlassian.net/browse/TNL-3126\r\n\r\n**Background:** The teams tests took twice as long to execute with the addition on the indexing signal.  Noted here: https://github.com/edx/edx-platform/pull/9365#issuecomment-134634258 . This PR creates a shared helper function that can be used to skip a signal and applies it to the teams `post_save` callback.\r\n\r\n**Studio Updates:** None.\r\n\r\n**LMS Updates:** Teams tests should run in a reasonable amount of time again.\r\n\r\n**Pre commit local test time**\r\n```\r\n.............................................................................\r\n.............................................................................\r\n.....................................\r\n-----------------------------------------------------------------------------\r\n191 tests run in 503.8 seconds (191 tests passed)\r\n```\r\n\r\n**Post commit local test time**\r\n```\r\n.............................................................................\r\n.............................................................................\r\n.....................................\r\n-----------------------------------------------------------------------------\r\n191 tests run in 269.1 seconds (191 tests passed)\r\n```"},"comment":{"url":"https://api.github.com/repos/edx/edx-platform/issues/comments/135150403","html_url":"https://github.com/edx/edx-platform/pull/9489#issuecomment-135150403","issue_url":"https://api.github.com/repos/edx/edx-platform/issues/9489","id":135150403,"user":{"login":"ormsbee","id":41625,"avatar_url":"https://avatars.githubusercontent.com/u/41625?v=3","gravatar_id":"","url":"https://api.github.com/users/ormsbee","html_url":"https://github.com/ormsbee","followers_url":"https://api.github.com/users/ormsbee/followers","following_url":"https://api.github.com/users/ormsbee/following{/other_user}","gists_url":"https://api.github.com/users/ormsbee/gists{/gist_id}","starred_url":"https://api.github.com/users/ormsbee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ormsbee/subscriptions","organizations_url":"https://api.github.com/users/ormsbee/orgs","repos_url":"https://api.github.com/users/ormsbee/repos","events_url":"https://api.github.com/users/ormsbee/events{/privacy}","received_events_url":"https://api.github.com/users/ormsbee/received_events","type":"User","site_admin":false},"created_at":"2015-08-26T19:45:12Z","updated_at":"2015-08-26T19:45:12Z","body":"Nice. :-) I actually expected a somewhat higher impact... just curious, as an experiment, could you please try to have `course_team_post_save_callback()` or the indexing method itself fail badly (e.g. 1/0 or raise an exception) to see if there are other calls to it sneaking through when you run your tests?\r\n\r\nThank you!"}}