I have written test case for this trigger.but can anyone tell me where is wrong in this...

trigger copystreetaddress on JunctionObject__c (be​fore insert)
{
      
    if(trigger.isInsert)
    {
        Set<Id> accountId = new Set<Id>();
        Set<Id> contactId = new Set<Id>();
        for(JunctionObject__c tempJ : Trigger.new)
        {
            accountId.add(tempJ.Account__c);
            contactId.add(tempJ.Contact__c);
        }
        
        List<Account> AccountList = [Select id,Nam​e,BillingStreet from Account where id in: accountI​d];
        
        List<Contact> ContactList = [Select id,Nam​e,OtherStreet from Contact where id in: contactId]​;
        
        for(JunctionObject__c tempJ : Trigger.new)
        {
            for(Account tempAcc : AccountList)
            {
                if(tempJ.Account__c == tempAcc.Id)
                {
                    for(Contact tempCon : ContactL​ist)
                    {
                        if(tempJ.Contact__c == tem​pCon.Id)
                        {
                            tempCon.OtherStreet = ​tempAcc.BillingStreet;
                        }                   
                    
                    }
                
                }
            
            }   
            
        }       
            update ContactList;
    }
       
}

controller method

.......................................

public class VfController{
                private final Contact con;
                
        public VfController(){
        con=[select id,name,Account.name,Account.id,Account.billingstreet from contact where id=:
           Apexpages.currentpage().getParameters().get('id')];
}
     public contact getCon(){
     return con;
     }
     public pageReference save(){
      update con;
      return null;
      }
      }