Skip to content

Commit

Permalink
Standardize the access to the create_random_item method
Browse files Browse the repository at this point in the history
  • Loading branch information
Tim Almdal committed Aug 29, 2009
1 parent 08d7fda commit 27b8125
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions modules/gallery/tests/Item_Model_Test.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@
*/
class Item_Model_Test extends Unit_Test_Case {
public function saving_sets_created_and_updated_dates_test() {
$item = self::create_random_item();
$item = self::_create_random_item();
$this->assert_true(!empty($item->created));
$this->assert_true(!empty($item->updated));
}

private function create_random_item() {
private static function _create_random_item() {
$item = ORM::factory("item");
/* Set all required fields (values are irrelevant) */
$item->name = rand();
Expand All @@ -33,7 +33,7 @@ private function create_random_item() {
}

public function updating_doesnt_change_created_date_test() {
$item = self::create_random_item();
$item = self::_create_random_item();

// Force the creation date to something well known
$db = Database::instance();
Expand All @@ -47,7 +47,7 @@ public function updating_doesnt_change_created_date_test() {
}

public function updating_view_count_only_doesnt_change_updated_date_test() {
$item = self::create_random_item();
$item = self::_create_random_item();
$item->reload();
$this->assert_same(0, $item->view_count);

Expand All @@ -64,7 +64,7 @@ public function updating_view_count_only_doesnt_change_updated_date_test() {

public function move_photo_test() {
// Create a test photo
$item = self::create_random_item();
$item = self::_create_random_item();

file_put_contents($item->thumb_path(), "thumb");
file_put_contents($item->resize_path(), "resize");
Expand Down Expand Up @@ -128,7 +128,7 @@ public function move_album_test() {

public function item_rename_wont_accept_slash_test() {
// Create a test photo
$item = self::create_random_item();
$item = self::_create_random_item();

$new_name = rand() . "/";

Expand All @@ -142,7 +142,7 @@ public function item_rename_wont_accept_slash_test() {
}

public function save_original_values_test() {
$item = $this->create_random_item();
$item = self::_create_random_item();
$item->title = "ORIGINAL_VALUE";
$item->save();
$item->title = "NEW_VALUE";
Expand Down

0 comments on commit 27b8125

Please sign in to comment.