function readOnly(count){ }
Starting November 20, the site will be set to read-only. On December 4, 2023,
forum discussions will move to the Trailblazer Community.
+ Start a Discussion
jaanvivekjaanvivek 

Unit Testing for Apex

Please make me correct for writing unit test class for the following Apex class-

 

public class practiseOneController
 {
    public Account getAccount()
    {
        Account[] acc= [select id,Name,(select id,firstname,lastname from Contacts limit 5) from Account where id=
                 :System.currentPageReference().getParameters().get('id')];
                 
                 if(acc.size()>0)
                 {
                    return acc[0];
                 
                 }
                  else
                  {
                    return null;
                  
                  }
                 
                 
    }
    
    
    // Test Method for this main class
    
     static testMethod void TestpractiseOneController()
     {
       practiseOneController testObj= new practiseOneController();
       
       Account testAcc=new Account();
       testAcc.Name='testaccount';
       insert testAcc;
       testObj.getAccount();
     
     
     
     }
    
    
    

}

 I tried but not getting to cover

 :System.currentPageReference().getParameters().get('id')];
                 
                 if(acc.size()>0)
                 {
                    return acc[0];
                 
                 }

 In this apex class i am trying to show the accociated contacts with the Account.

 

Thanks & regards,

JaanVivek

Best Answer chosen by Admin (Salesforce Developers) 
Anoop AsokAnoop Asok

Hi,

Please replace str=testAcc.Id; with testObj.id = testAcc.id; and str=null; with testObj.id = null;

 

Thanks,

Anoop Asok

All Answers

Anoop AsokAnoop Asok

Hi,

This code is not getting covered since you're trying to extract the id value from the page reference which doesn't exist.

 

Workaround

-----------------

1. Create a public variable to hold the id value.

2. Write a constructor and set the id value to System.currentPageReference().getParameters().get('id')

3. Update your test method to insert some dummy Account records, since by default during test execution none of the existing data in the env will be visible to the logic.

4. Set the value of the public variable created on step 1 with the id of one of the dummy account and contact records created on step 3.

5. Invoke getAccount() method.

6. delete the dummy account records OR reset id value to null.

7. invoke getAccount() method again.

 

This will probably give you 100% coverage.

 

Thanks,

Anoop Asok

jaanvivekjaanvivek

Hi,

 

I  have tried as below but still it's lacking to get some data in Account[].

 

public class practiseOneController
 {
     public Id id;
     
     public practiseOneController()
      {
         id=System.currentPageReference().getParameters().get('id');
         
      }
   
    public Account getAccount()
    {
        Account[] acc= [select id,Name,(select id,firstname,lastname from Contacts limit 5) from Account where id=
                 :id];
                 
                 if(acc.size()>0)
                 {
                    return acc[0];
                 
                 }
                  else
                  {
                    return null;
                  
                  }
                 
                 
    }
    
    
    // Test Method for this main class
    
     static testMethod void TestpractiseOneController()
     {
       practiseOneController testObj= new practiseOneController();
       
       Id str;
       Account testAcc=new Account();
       testAcc.Name='testaccount';
       insert testAcc;
       str=testAcc.Id;
       testObj.getAccount();
       str=null;
       testObj.getAccount(); 
      
     
     
     }
    
    
    

}

 Please suggest in this.

 

 

Thanks & regards,

JaanVivek

Anoop AsokAnoop Asok

Hi,

Please replace str=testAcc.Id; with testObj.id = testAcc.id; and str=null; with testObj.id = null;

 

Thanks,

Anoop Asok

This was selected as the best answer