{"action":"created","issue":{"url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/16","labels_url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/16/comments","events_url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/16/events","html_url":"https://github.com/mbotos/SmartFactory-for-Force.com/issues/16","id":15102118,"number":16,"title":"No assert in loadTestWith100PlusAccountNContacts test method","user":{"login":"mattlacey","id":1256283,"avatar_url":"https://avatars.githubusercontent.com/u/1256283?v=3","gravatar_id":"","url":"https://api.github.com/users/mattlacey","html_url":"https://github.com/mattlacey","followers_url":"https://api.github.com/users/mattlacey/followers","following_url":"https://api.github.com/users/mattlacey/following{/other_user}","gists_url":"https://api.github.com/users/mattlacey/gists{/gist_id}","starred_url":"https://api.github.com/users/mattlacey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattlacey/subscriptions","organizations_url":"https://api.github.com/users/mattlacey/orgs","repos_url":"https://api.github.com/users/mattlacey/repos","events_url":"https://api.github.com/users/mattlacey/events{/privacy}","received_events_url":"https://api.github.com/users/mattlacey/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2013-06-04T05:04:59Z","updated_at":"2015-06-08T15:25:38Z","closed_at":"2015-06-08T15:25:38Z","body":"Test methods with no asserts get flagged when submitting applications for security review, so I swapped the method out for the following:\r\n```apex\r\n    static testMethod void loadTestWith100PlusAccountNContacts() {\r\n\r\n    \tList<Account> accounts = new List<Account>();\r\n    \tList<Contact> contacts = new List<Contact>();\r\n\r\n    \tfor (Integer idx = 0; idx <= 101; idx++)\r\n    \t\taccounts.Add((Account)SmartFactory.createSObject('Account'));\r\n\r\n\t\tfor (Integer idx = 0; idx <= 101; idx++)\r\n    \t\tcontacts.Add((Contact)SmartFactory.createSObject('Contact'));\r\n\r\n    \tinsert accounts;\r\n    \tinsert contacts;\r\n\r\n    \tSystem.Assert([select Id from Contact].Size() == 102 && [select Id from Account].Size() == 102);\r\n    }\r\n```"},"comment":{"url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/comments/110039954","html_url":"https://github.com/mbotos/SmartFactory-for-Force.com/issues/16#issuecomment-110039954","issue_url":"https://api.github.com/repos/mbotos/SmartFactory-for-Force.com/issues/16","id":110039954,"user":{"login":"mbotos","id":737005,"avatar_url":"https://avatars.githubusercontent.com/u/737005?v=3","gravatar_id":"","url":"https://api.github.com/users/mbotos","html_url":"https://github.com/mbotos","followers_url":"https://api.github.com/users/mbotos/followers","following_url":"https://api.github.com/users/mbotos/following{/other_user}","gists_url":"https://api.github.com/users/mbotos/gists{/gist_id}","starred_url":"https://api.github.com/users/mbotos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbotos/subscriptions","organizations_url":"https://api.github.com/users/mbotos/orgs","repos_url":"https://api.github.com/users/mbotos/repos","events_url":"https://api.github.com/users/mbotos/events{/privacy}","received_events_url":"https://api.github.com/users/mbotos/received_events","type":"User","site_admin":false},"created_at":"2015-06-08T15:25:38Z","updated_at":"2015-06-08T15:25:38Z","body":"Since SmartFactory wasn't designed for bulk data creation, I've removed the bulk unit tests."}}